4 ms·

As simplistic as it may sound but Google probably removed it mostly because it's a chunk of c++ that they don't want around. I'm the last person to usually endo

As simplistic as it may sound but Google probably removed it mostly because it's a chunk of c++ that they don't want around. I'm the last person to usually endorse a rewrite but in this case maybe they should consider a Rust port so browsers feel more comfortable including this. The point of an image format is to be adopted by everyone.. it's kind of beneficial if you make that usecase as attractive as possible. Personally I think Rust is actually hard to understand and for that reason dangerous too. But it definitely has some safety over c++.

lifthrasiir 4y ago
Chromium is an extremely large C++ code base and I don't think they would reject libjxl only for that reason. That said, as an author of the first significant reimplementation of JPEG XL format (J40), I think a Rust decoder would be a good addition to the format and has a long-term plan to make one [1]. [1] https://github.com/lifthrasiir/j40/issues/10 https://github.com/lifthrasiir/j40/issues/10
- ZeroGravitas 4y ago
  Some on the Mozilla side did raise this as a specific issue. One interesting project would be a combined JPEG and JPEG XL decoder, written in rust which might even lead to a reduction in code size and better security since they have some overlap.
hypothesis 4y ago
This is a valid argument, but which of the contenders are using Rust or similar security minded languages? WebP appears to be written in C++, perhaps because underlying video codec is also written in C++…
- JyrkiAlakuijala 4y ago
  We wrote WebP in C -- in the creative phase of codec and format development I kept it in C++, the feasibility proto was initially launched in C++ (perhaps 0.2 version of the lossless coding, late 2011), but I ported it to C a few months after that and handed it over to the WebP production team (early 2012) who kept making improvements like streaming decoding
  - hypothesis 4y ago
    My bad, I glanced at “compiling” page and assumed c++ for some reason. [0] https://developers.google.com/speed/webp/docs/compiling https://developers.google.com/speed/webp/docs/compiling
- lifthrasiir 4y ago
  There is a Rust WebP decoder in image-rs [1], presumably independently written from the specification. [1] https://github.com/image-rs/image/tree/master/src/codecs/webp https://github.com/image-rs/image/tree/master/src/codecs/web...
  - est31 4y ago
    FTR that decoder does not support colors, so for most uses it is not ready yet: https://github.com/image-rs/image/pull/952 https://github.com/image-rs/image/pull/952
cercatrova 4y ago
Isn't that why they're creating Carbon?

lifthrasiir 4y ago

Chromium is an extremely large C++ code base and I don't think they would reject libjxl only for that reason. That said, as an author of the first significant reimplementation of JPEG XL format (J40), I think a Rust decoder would be a good addition to the format and has a long-term plan to make one [1]. [1] https://github.com/lifthrasiir/j40/issues/10 https://github.com/lifthrasiir/j40/issues/10

ZeroGravitas 4y ago
Some on the Mozilla side did raise this as a specific issue. One interesting project would be a combined JPEG and JPEG XL decoder, written in rust which might even lead to a reduction in code size and better security since they have some overlap.

hypothesis 4y ago

This is a valid argument, but which of the contenders are using Rust or similar security minded languages? WebP appears to be written in C++, perhaps because underlying video codec is also written in C++…

JyrkiAlakuijala 4y ago
We wrote WebP in C -- in the creative phase of codec and format development I kept it in C++, the feasibility proto was initially launched in C++ (perhaps 0.2 version of the lossless coding, late 2011), but I ported it to C a few months after that and handed it over to the WebP production team (early 2012) who kept making improvements like streaming decoding
- hypothesis 4y ago
  My bad, I glanced at “compiling” page and assumed c++ for some reason. [0] https://developers.google.com/speed/webp/docs/compiling https://developers.google.com/speed/webp/docs/compiling
lifthrasiir 4y ago
There is a Rust WebP decoder in image-rs [1], presumably independently written from the specification. [1] https://github.com/image-rs/image/tree/master/src/codecs/webp https://github.com/image-rs/image/tree/master/src/codecs/web...
- est31 4y ago
  FTR that decoder does not support colors, so for most uses it is not ready yet: https://github.com/image-rs/image/pull/952 https://github.com/image-rs/image/pull/952

cercatrova 4y ago

Isn't that why they're creating Carbon?