This looks better, but I still think you should use WICPersistOptionNoCacheStream.
Some of your failure cases in create_metadata_reader will return a reference to an IWICMetadataReader, which seems wrong.
Vincent Povirk madewokherd@gmail.com wrote:
This looks better, but I still think you should use WICPersistOptionNoCacheStream.
Some of your failure cases in create_metadata_reader will return a reference to an IWICMetadataReader, which seems wrong.
Apparently that was a hasty attempt to make the patch appear in today's commit round. Sorry about that, I'll resend a fixed version tomorrow.