[prev in list] [next in list] [prev in thread] [next in thread] 

List:       wine-devel
Subject:    Re: [PATCH 1/9 v3] gdiplus: Add partial support for GdipDrawImagePointsRect on metafile
From:       Piotr Caban <piotr.caban () gmail ! com>
Date:       2017-06-30 15:10:43
Message-ID: f2f854ee-b303-47e4-d856-52cbf6f31b53 () gmail ! com
[Download RAW message or body]

On 06/30/17 16:46, Dmitry Timoshkov wrote:
> Piotr Caban <piotr.caban@gmail.com> wrote:
> 
>>> Even internally gdiplus shouldn't do this kind of hacks IMO, if there is
>>> a need for a shortcut for a particular encoder class id it's better to
>>> add a helper. Have a look at the encoder/decoder tests how it's supposed
>>> to look like.
>> I've done it this way because I don't think we should go through
>> enumerating all supported formats internally.
> 
> Why not?
Mainly because it unnecessarily complicates the code. Also I don't see 
any advantages from doing it this way.



[prev in list] [next in list] [prev in thread] [next in thread]