mbien commented on PR #9278:
URL: https://github.com/apache/netbeans/pull/9278#issuecomment-4071303848

   > There were other instances that could be replaced with 
`InputStream#readAllBytes()`,
   
   I revisited the code section where the method was inlined and rewrote some 
of the usages as simplified ARM blocks, using `readAllBytes()`, `write(bytes)` 
and `transferTo()`. This removed about 700 lines.
   
   I am not really expecting anyone reviewing via github this since the 
changeset is now larger. Might be easier via IDE. I will take another good look 
myself before merging.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists

Reply via email to