Performance issue with the redundancy operations

[yyzdtccjdtc]

rust-term-grid/src/lib.rs

Line 319 in 1cbc10f

widths[index] = max(widths[index], cell.width);

For this line of code, it will always update the value of widths[index], even if the value of it does not change. This introduces some redundant operations.

If we change the code to this, we can eliminate these redundant operations.

if widths[index] < cell.width{
     widths[index] = cell.width;
}

According to my tests, the average execution time decreased from 6.5s to 4.8s, which is a 1.37x speedup.

Hope this information helps!

[pengfei-su]

@ogham
Is this repo still maintained?