image-rs / image-tiff

TIFF decoding and encoding library in pure Rust

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

JPEG Compression Tag

powturns opened this issue · comments

I noticed that CompressionMethod::JPEG has the value 6. According to wikipedia, this is actually the "obsolete 'old-style' JPEG" that "... should never be written". For consistency with Deflate/OldDeflate should we consider renaming this to OldJPEG?

I realize this is a breaking change, but since it seems like there is some work being done around this area it may be a good opportunity to standardize the names.

I would just deprecate the old variant. It matches the intended semantics and is not a breaking change.

If we incorporate the modern JPEG (7), would that be called NewJPEG?

I prefer the implicit suggestion, ModernJPEG 😀