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.