Re: [3/4] windowscodecs: Implement CreateBitmapFromMemory.

2013-01-02 Thread Dmitry Timoshkov
Vincent Povirk wrote: > I guess I prefer to keep interfaces as simple as possible even if it > means writing more code? I'd consider writing a helper function, but > none of the other methods would use it. Is there any hurry with CreateBitmapFromMemory implementation, or you somehow prefer your

Re: [3/4] windowscodecs: Implement CreateBitmapFromMemory.

2013-01-02 Thread Vincent Povirk
On Wed, Jan 2, 2013 at 11:42 PM, Dmitry Timoshkov wrote: > It's better to actually check return value of IWICBitmapLock_GetDataPointer > before memecpy(), or if that's not supposed to fail drop 'hr' assignment. Whoops, good catch. (I don't think it can fail, but I didn't mean to make that assumpt

Re: [3/4] windowscodecs: Implement CreateBitmapFromMemory.

2013-01-02 Thread Dmitry Timoshkov
Vincent Povirk wrote: > +hr = IWICBitmap_Lock(bitmap, NULL, WICBitmapLockWrite, &lock); > +if (SUCCEEDED(hr)) > +{ > +UINT buffersize; > +BYTE *buffer; > + > +hr = IWICBitmapLock_GetDataPointer(lock, &buffersize, &buffer); > + > +

Re: [3/4] windowscodecs: Implement CreateBitmapFromMemory.

2013-01-02 Thread Marvin
Hi, While running your changed tests on Windows, I think I found new failures. Being a bot and all I'm not very good at pattern recognition, so I might be wrong, but could you please double-check? Full results can be found at http://testbot.winehq.org/JobDetails.pl?Key=23710 Your paranoid android