Formatting and isolating rayon feature completely

image-rs / jpeg-decoder

JPEG decoder written in Rust

Apache License 2.0

150 stars 87 forks source link

Formatting and isolating rayon feature completely #240

Closed gents83 closed 2 years ago

gents83 commented 2 years ago

@HeroicKatora this is actually fully separating rayon to make it work on wasm target. I had other modification in local I didn't integrate in my branch before merging - my bad :( This will also require to update current release version probably

For the future me - remind to do submit from your pc and not from mobile!!!

gents83 commented 2 years ago

Is there any way we can test this is CI? My confidence in future changes preserving the wasm-functionality is a bit low. Apart from the nit, looks good.

Mmmm... it'll really depends in what will be added in rayon module little by little, no? Let's say that the best is to do what you're suggesting - to put all the rayon-related stuff inside rayon.rs - no?

HeroicKatora commented 2 years ago

Release when the testing as detailed in #241 lands on CI, eta tomorrow.