Re: [4/4] windowscodecs: Implement IWICComponentFactory::CreateDecoderFromFileHandle.

[Vincent Povirk]

> - hr = map_file(file, &map, &mem, &size);
> - if (FAILED(hr)) return hr;
> -

...
> -
> - hr = IWICStreamImpl_InitializeFromMemory(stream, mem, size.u.LowPart);
> - if (FAILED(hr)) goto error;

This makes me sad.

Is this really simpler than implementing Write and Seek for a file handle?

[4/4] windowscodecs: Implement IWICComponentFactory::CreateDecoderFromFileHandle.

[Hans Leidekker]

On Wed, 2013-02-06 at 10:25 -0600, Vincent Povirk wrote:
> > - hr = map_file(file, &map, &mem, &size);
> > - if (FAILED(hr)) return hr;
> > -
>
> ...
> > -
> > - hr = IWICStreamImpl_InitializeFromMemory(stream, mem, size.u.LowPart);
> > - if (FAILED(hr)) goto error;
>
> This makes me sad.
>
> Is this really simpler than implementing Write and Seek for a file handle?

I'd say it's a comparable amount of code. I don't have a strong preference for
either option.