Thomas, can you please review this, discuss it with Mario and give me your assessment. And of course if you have a patch, please attach it to this issue. As Mario has been kind enough to provide a test file, we should also consider updating the test suite to take advantage of that file.
Yes, patch works as expected. I could not apply the provided Exif.Image.PageNumber.patch but needed to manually patch it. New patch T1044.patch should hopefully work.
Thank you, Mario and Robin!
Thanks for the great (and very fast) work, Thomas and Robin! Two thumbs up!
Submitted r3632. Thank You, Mario for reporting this and providing the patch. Thank You, Thomas for validating the patch.
- Assignee changed from Thomas Beutlich to Robin Mills
Back to Robin for the optional test case. I have no idea how to do it since I do not run the test suite.
Is there any reason why we cannot copy your test file into our test suite (for example copyright). If you're OK with this, I'll add your file to the test suite tomorrow as a regression detector.
Dear Robin and all, yes I agree, please use the file I sent in your test collection and anywhere else in development where you see fit! No copyright restrictions apply (as long as the predominant use is to aid in your development).
Thanks again and all the best!
- Status changed from Assigned to Resolved
Thanks Mario. Test suite updated: r3640. Good teamwork. Thanks Everybody.
- % Done changed from 0 to 100
- Status changed from Resolved to Closed
Also available in: Atom