From: Lutz <lu...@to...> - 2006-01-25 21:29:28
|
On Wed, 2006-01-25 at 10:03 -0500, Hubert Figuiere wrote: > > I'd prefer another option EXIF_DATA_OPTION_INCLUDING_JPEG_HEADER (or > > similar name) that is set by default. That way, the API remains more > > compact. >=20 > OK. But it brings a bit more complexity IMHO No complexity in the code: void exif_data_save_data ()=20 { if (option & INCLUDING_JPEG_HEADER) _exif_data_save_data (..., 1) else _exif_data_save_data (..., 0) } Complexity in the API? I don't think so. > > The exif loader can set the flag according to the data it got. > > No need for frontends to call an additional function then. >=20 > Set the output automatically, I don't think that is a good idea as I=20 > know some people already work around that problem. That would break the A= PI. Agreed. --=20 Lutz M=FCller <lu...@to...> |