Re: RFR: 8299199: Avoid redundant split calls in FontConfiguration.initReorderMap implementations

2022-12-21 Thread Alexey Ivanov
On Wed, 21 Dec 2022 11:56:52 GMT, Andrey Turbanov wrote: >> src/java.desktop/share/classes/sun/awt/FontConfiguration.java line 856: >> >>> 854: >>> 855: protected String[] split(String sequence) { >>> 856: Vector v = splitSequence(sequence); >> >> Can `splitSequence` be replaced

Re: RFR: 8299199: Avoid redundant split calls in FontConfiguration.initReorderMap implementations

2022-12-21 Thread Alexey Ivanov
On Wed, 21 Dec 2022 11:03:08 GMT, Andrey Turbanov wrote: > `FontConfiguration.split` uses legacy synchronized `Vector` class. But > actually there is no need to use it, because input arguments are known at > compile time. Instead of split usages we can just create String[] array with > filled

Re: RFR: 8299199: Avoid redundant split calls in FontConfiguration.initReorderMap implementations

2022-12-21 Thread Andrey Turbanov
On Wed, 21 Dec 2022 11:35:47 GMT, Alexey Ivanov wrote: >> `FontConfiguration.split` uses legacy synchronized `Vector` class. But >> actually there is no need to use it, because input arguments are known at >> compile time. Instead of split usages we can just create String[] array with >>

Re: RFR: 8299199: Avoid redundant split calls in FontConfiguration.initReorderMap implementations

2022-12-21 Thread Alexey Ivanov
On Wed, 21 Dec 2022 11:03:08 GMT, Andrey Turbanov wrote: > `FontConfiguration.split` uses legacy synchronized `Vector` class. But > actually there is no need to use it, because input arguments are known at > compile time. Instead of split usages we can just create String[] array with > filled

RFR: 8299199: Avoid redundant split calls in FontConfiguration.initReorderMap implementations

2022-12-21 Thread Andrey Turbanov
`FontConfiguration.split` uses legacy synchronized `Vector` class. But actually there is no need to use it, because input arguments are known at compile time. Instead of split usages we can just create String[] array with filled values directly. - Commit messages: - [PATCH] Avoid