Exif data not parsed for specific images about exifreader HOT 7 CLOSED

I created a PR, please take a look, at very least I'll have to add a few tests...

from exifreader.

Seems this jpeg uses JFIF, not sure if this is something this library is supposed to support?

from exifreader.

Hi! No problem with JFIF, however, from what I understand most of those fields are from the ICC color profile which is not supported. I haven't looked at it enough to know how much work it would be to implement it, but I guess it would be possible. Will have a deeper look later on.

from exifreader.

@mattiasw I mostly added ICC profile parsing to this library, let me know if you would be interesting in accepting PR with these changes.

Current work is here (in master, I'll squash commits later): https://github.com/bh213/ExifReader/commits/master

I still have to refactor tags parsing into separate class with better names, add a few tests & update typescript definitions.

from exifreader.

Nice! I would definitely be interested in a PR for those changes.

from exifreader.

I'll try to have a look over the weekend.

from exifreader.

Implemented in #59.

from exifreader.