kwin commented on PR #29:
URL: 
https://github.com/apache/sling-org-apache-sling-graphql-core/pull/29#issuecomment-1188253002

   `close()` is the last method called on an instance of try with resources so 
I fail to see how this would have a negative impact. The CloseShieldWriter uses 
that to detect in case close is called twice, but it won’t have an effect on 
either the wrapped reader nor on any wrappers of CloseShieldWriter.


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

Reply via email to