[GitHub] [commons-imaging] gwlucastrig commented on pull request #116: [IMAGING-159] Add ImagingParameters interface and BaseParameters (POJO)

[GitBox <gi...@apache.org>]

gwlucastrig commented on pull request #116:
URL: https://github.com/apache/commons-imaging/pull/116#issuecomment-961784862


   My perspective is limited because I am so focused on TIFFs. And I seldom use the basic Imaging class (preferring the TIFF-specific parser and write classes).   So I am sure I am unaware of a number of valid use cases. 
   
   That being said, I might think that simpler is better.  The Imaging class API could be cut back to just include only limited support for parameters.  The read-related operations might not even take parameters arguments.  The write-related would only take a very basic parameters.  In fact, except for a "dots per inch" or "mm per pixel" setting, I can't think of anything that would be supported across the majority of write operations.  


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@commons.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org