RFR: JDK-8297412: Remove easy warnings in javafx.fxml, javafx.media, javafx.swing, javafx.swt and javafx.web [v4]
Kevin Rushforth
kcr at openjdk.org
Tue Nov 22 19:12:30 UTC 2022
On Tue, 22 Nov 2022 18:35:36 GMT, Andy Goryachev <angorya at openjdk.org> wrote:
>> John Hendrikx has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fix indent of 5 spaces to 4 spaces
>
> modules/javafx.fxml/src/main/java/javafx/fxml/FXMLLoader.java line 1446:
>
>> 1444: @Override
>> 1445: public void processEndElement() throws IOException {
>> 1446: super.processEndElement();
>
> this suggests a specific pattern, even though there is probably no development expected in this class.
> i'd keep it as is, and revert making the method abstract in the base class.
I agree. This sort of change goes beyond what I expect to see in a cleanup pass.
-------------
PR: https://git.openjdk.org/jfx/pull/958
More information about the openjfx-dev
mailing list