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.