Re: RFR: 8284796: sun.security.ssl.Finished::toString misses a line feed in the message format pattern

2022-04-14 Thread John Jiang
On Thu, 14 Apr 2022 18:08:21 GMT, Xue-Lei Andrew Fan wrote: >> The log for Finished message looks like the below, >> >> "Finished": { >> "verify data": { >> : ... ... >> }'} // looks weird >> >> because a line feed is missing in the format pattern. >> >> ""Finished": '{'\n" + >>

Re: RFR: 8284796: sun.security.ssl.Finished::toString misses a line feed in the message format pattern

2022-04-14 Thread Xue-Lei Andrew Fan
On Wed, 13 Apr 2022 05:10:22 GMT, John Jiang wrote: > The log for Finished message looks like the below, > > "Finished": { > "verify data": { > : ... ... > }'} // looks weird > > because a line feed is missing in the format pattern. > > ""Finished": '{'\n" + > " "verify data":

Re: RFR: 8284796: sun.security.ssl.Finished::toString misses a line feed in the message format pattern

2022-04-14 Thread Xue-Lei Andrew Fan
On Wed, 13 Apr 2022 07:14:04 GMT, John Jiang wrote: > Could I not change this style? Sure. But effort of this update will be overrode if text blocks are used. I would suggest to use text block once the code block get touched. - PR: https://git.openjdk.java.net/jdk/pull/8215

Re: RFR: 8284796: sun.security.ssl.Finished::toString misses a line feed in the message format pattern

2022-04-13 Thread John Jiang
On Wed, 13 Apr 2022 06:07:43 GMT, Xue-Lei Andrew Fan wrote: >> The log for Finished message looks like the below, >> >> "Finished": { >> "verify data": { >> : ... ... >> }'} // looks weird >> >> because a line feed is missing in the format pattern. >> >> ""Finished": '{'\n" + >>

Re: RFR: 8284796: sun.security.ssl.Finished::toString misses a line feed in the message format pattern

2022-04-13 Thread Xue-Lei Andrew Fan
On Wed, 13 Apr 2022 05:10:22 GMT, John Jiang wrote: > The log for Finished message looks like the below, > > "Finished": { > "verify data": { > : ... ... > }'} // looks weird > > because a line feed is missing in the format pattern. > > ""Finished": '{'\n" + > " "verify data":

RFR: 8284796: sun.security.ssl.Finished::toString misses a line feed in the message format pattern

2022-04-12 Thread John Jiang
The log for Finished message looks like the below, "Finished": { "verify data": { : ... ... }'} // looks weird because a line feed is missing in the format pattern. ""Finished": '{'\n" + " "verify data": '{'\n" + "{0}\n" + " '}'" + // a line feed is needed "'}'", -