RFR: 8295729: Add jcheck whitespace checking for properties files [v3]
Alexey Ivanov
aivanov at openjdk.org
Sat Oct 29 12:14:25 UTC 2022
On Mon, 24 Oct 2022 19:29:41 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> Magnus Ihse Bursie has updated the pull request incrementally with two additional commits since the last revision:
>>
>> - Revert "Remove check for .properties from jcheck"
>>
>> This reverts commit c91fdaa19dc06351598bd1c0614e1af3bfa08ae2.
>> - Change trailing space and tab in values to unicode encoding
>
> src/java.sql.rowset/share/classes/com/sun/rowset/RowSetResourceBundle.properties line 134:
>
>> 132:
>> 133: #SyncResolverImpl exceptions
>> 134: syncrsimpl.indexval = Index value out of range\u0020\u0020
>
> prob. unnecessary
This case is similar the one you mentioned below. If this value is used in a string template or concatenation, the trailing white-space could be necessary; however, one space is probably enough.
-------------
PR: https://git.openjdk.org/jdk/pull/10792
More information about the compiler-dev
mailing list