Re: RFR: 8307077: Convert CRLF to LF in java.xml.crypto

2023-04-28 Thread Sean Mullan
On Thu, 27 Apr 2023 19:01:07 GMT, Weijun Wang wrote: > Convert CRLF to LF in these files so it's consistent with other source files. Marked as reviewed by mullan (Reviewer). - PR Review: https://git.openjdk.org/jdk/pull/13700#pullrequestreview-1405976589

Re: RFR: 8307077: Convert CRLF to LF in java.xml.crypto

2023-04-27 Thread Weijun Wang
On Thu, 27 Apr 2023 23:06:40 GMT, Mark Powers wrote: >> Convert CRLF to LF in these files so it's consistent with other source files. > > LGTM > I wrote a program to count CRLF in the four changed files in order to verify > your fix. @mcpowers I looked at the lower-right corner of IntelliJ IDEA

Re: RFR: 8307077: Convert CRLF to LF in java.xml.crypto

2023-04-27 Thread Mark Powers
On Thu, 27 Apr 2023 19:01:07 GMT, Weijun Wang wrote: > Convert CRLF to LF in these files so it's consistent with other source files. LGTM I wrote a program to count CRLF in the four changed files in order to verify your fix. - PR Review: https://git.openjdk.org/jdk/pull/13700#pull