You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@commons.apache.org by GitBox <gi...@apache.org> on 2021/09/09 02:22:46 UTC

[GitHub] [commons-imaging] gwlucastrig commented on pull request #165: IMAGING-266 read numeric data from GeoTIFFs

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


   Sorry, I forgot about the Spotbugs test (did checkstyle, pmd, and findbugs, but not Spotbugs).  So I just submitted changes for Spotbugs.  
   
   For reasons of efficiency, the TiffRasterData classes expose arrays directly through the API. This approach is reasonable because the raster data classes are really just simple containers for transferring image read results to the calling application.  Also, since many TIFF rasters consist of literally 100s of millions of raster cells, we wish to avoid making redundant copies of data in memory.  This rationale is described in the Javadoc for the relevant methods with appropriate caveats for developers.


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