[GitHub] [commons-imaging] gwlucastrig commented on pull request #318: [IMAGING-316] Add support for BigTIFF format

2023-09-27 Thread via GitHub


gwlucastrig commented on PR #318:
URL: https://github.com/apache/commons-imaging/pull/318#issuecomment-1738274426

   Added Javadoc and corrected formatting.  In order to support an additional 
JUnit test, I needed to add a new toBytes() method for longs.  This method 
wasn't required for reading BigTIFFs, but it was necessary to implement the 
JUnit test.  It will be useful in the future for any implementation that writes 
BigTIFFs (the current implementation only reads them).



-- 
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-unsubscr...@commons.apache.org

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



[GitHub] [commons-imaging] gwlucastrig commented on pull request #318: Imaging-316: Add support for BigTIFF format

2023-09-25 Thread via GitHub


gwlucastrig commented on PR #318:
URL: https://github.com/apache/commons-imaging/pull/318#issuecomment-1734616510

   I hope I did this right.  Please let me know if it didn't work. 
   
   I am not adept at Git and was unable to find a clear set of instructions for 
submitting changes in response to a PR review. I'm sure they're out there, but 
I didn't find them.


-- 
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-unsubscr...@commons.apache.org

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



[GitHub] [commons-imaging] gwlucastrig commented on pull request #318: Imaging-316: Add support for BigTIFF format

2023-09-19 Thread via GitHub


gwlucastrig commented on PR #318:
URL: https://github.com/apache/commons-imaging/pull/318#issuecomment-1725688908

   Will address these comments and update code.


-- 
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-unsubscr...@commons.apache.org

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



[GitHub] [commons-imaging] gwlucastrig commented on pull request #318: Imaging-316: Add support for BigTIFF format

2023-09-19 Thread via GitHub


gwlucastrig commented on PR #318:
URL: https://github.com/apache/commons-imaging/pull/318#issuecomment-1725312015

   Do you have a feel for when this PR might get merged or, failing that, 
rejected?  I've got some other stuff I'm working on and am trying to plan for 
the next few weeks.


-- 
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-unsubscr...@commons.apache.org

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