Re: [PATCH v3 6/6] wmphoto: Implement WmpDecoder using jxrlib.

[Rémi Bernon]

On 2020-09-21 21:57, Esme Povirk (they/them) wrote:
> Fixed my issue with the old dll being used

Yeah it's a little bit annoying to have to delete the PE module, 
especially when editing the series or checking each patch. Maybe I 
should just remove the -no-cygwin flag first in the series.
> and now I get this:
> d3dx10.c:1380: Test succeeded inside todo block: Test 27: Got unexpected hr 0.
>
> It seems like wmp_stream_Read doesn't report the actual number of
> bytes read to the caller. How is it meant to report a short read?

I didn't know there were WMP tests elsewhere already, I'll have a look.
> I didn't realize you had a FIXME for multi-frame images elsewhere, we
> only need to report it once.

Alright.