dblevins commented on PR #84: URL: https://github.com/apache/johnzon/pull/84#issuecomment-1107921537
Can you check the code again? The tests don't use the second constructor and close is called only once. I'm not convinced trimming things like null to things like `n...` which is the same number of characters is a good idea. Do you have a scenario in mind where a user would actually find that more useful than seeing `null`? -- 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: dev-unsubscr...@johnzon.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
