SDWebImageAVIFCoder icon indicating copy to clipboard operation
SDWebImageAVIFCoder copied to clipboard

Handling ICC profile in another way.

Open ledyba-z opened this issue 5 years ago • 5 comments

Split from #19, #20.

Currently, we are discussing how should we treat ICC profile while encoding/decoding. (In here: AOMediaCodec/av1-avif#84)

After we agree about how should we do, I will rewrite the decoder and encoder according to it.

ledyba-z avatar May 13 '20 17:05 ledyba-z

@ledyba-z Can I release a patch version firstly, to fix the compile issue ?

This sounds need both libavif and SDWebImageAVIFCoder update, may be a minor version bump.

dreampiggy avatar May 14 '20 03:05 dreampiggy

@dreampiggy Yes, please release the new version.

This sounds need both libavif and SDWebImageAVIFCoder update, may be a minor version bump.

Yes, I think "v0.7.0" is better (However, it looks the breaking change in libavif is not released yet as v0.7.4 or v0.8.0).

ledyba-z avatar May 14 '20 08:05 ledyba-z

Seems the breaking change of profile constants was introduced during a patch version release 😅

image

libavif does not follows semversion currently. I can change that into s.dependency 'libavif', '>= 0.7.2' for this.

dreampiggy avatar May 14 '20 09:05 dreampiggy

Released v0.6.1 firstly. This compatible with libavif 0.7.2+

If libavif 0.8 or later do a huge refactory, we can bump into 0.7.0 again.

dreampiggy avatar May 14 '20 09:05 dreampiggy

Oh, sorry. You are right. I just saw only results of git blame :sweat_smile:

libavif does not follows semversion currently. I can change that into s.dependency 'libavif', '>= 0.7.2' for this. If libavif 0.8 or later do a huge refactory, we can bump into 0.7.0 again.

Both LGTM, thanks!

ledyba-z avatar May 14 '20 09:05 ledyba-z