From: "Rémi Bernon" Subject: Re: [PATCH v3 6/6] wmphoto: Implement WmpDecoder using jxrlib. Message-Id: <29246ad2-f8d1-1cc1-a57d-49195d3a750d@codeweavers.com> Date: Mon, 21 Sep 2020 22:03:29 +0200 In-Reply-To: References: <20200921172040.741741-1-rbernon@codeweavers.com> <20200921172040.741741-6-rbernon@codeweavers.com> 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. -- Rémi Bernon