W dniu 17.10.2010 01:45, Vincent Povirk pisze:
+1+ if (!prc) + { + UINT width, height; + hr = IWICBitmapSource_GetSize(This->source,&width,&height); + if (FAILED(hr)) return hr; + rc.X = 0; + rc.Y = 0; + rc.Width = width; + rc.Height = height; + prc =&rc; + } Any reason you didn't use this approach in the other functions?
I don't know really. Doing a memcpy to a local rectangle seems a more natural way to do it, as I don't really like to modify input arguments. In this case I guess I just blindly copied an existing solution that I found later during the review of the other codecs.
If you want I can unify the implementations so that all of them look like the one above.
