microsoft / lepton_jpeg_rust

Port of DropBox Lepton compression to Rust
Apache License 2.0
85 stars 9 forks source link

Logic simplification in `lepton_encode_row_range/decode_row_wrapper` #81

Open Melirius opened 1 month ago

Melirius commented 1 month ago

As mid_right is indistinguishable from middle and width_one from mid_left, logic inside these functions can be simplified, as well as lower in process_row and decode_row.

mcroomp commented 1 month ago

I think its vestiges of using the upper right block which never got implemented as far as I can tell, but yeah it can be dramatically simplified. Even one_width isn't necessary.

I had this change that I was working on #82