GITNE commented on issue #49: Add support for EXIF 2.32
URL: https://github.com/apache/commons-imaging/pull/49#issuecomment-505945725
 
 
   This PR does not make `ExifTagConstants` fully standards compliant though 
(unfortunately). There are still tag constant values left which do not comply 
with the data type as outlined in the EXIF specification, like 
`ExifTagConstants.EXPOSURE_PROGRAM_VALUE_MANUAL` which is of type `int` while 
it should be `short`. Updating their data types to the correct data types is 
easy but this change may break backwards compatibility for some applications. 
Simply deprecating them is no option either because their replacements would 
need to be labeled with new identifiers which ultimately would create new 
inconsistencies and unnecessary mess. So, I did not touch these since breaking 
backwards compatibility would require developer consensus, even though 
commons-imaging—if I am not mistaken—has not reached version 1.0 status yet.
   
   Should you have any idea how to fix these constants without breaking 
backwards compatibility, I would be happy to read about it.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to