RFR: 8269638: Property methods, setters, and getters in printing API should be final [v2]

[Prasanta Sadhukhan]

On Mon, 12 Jul 2021 22:04:25 GMT, Phil Race <prr at openjdk.org> wrote:

>> - Make various setters and getters and properties final as needed
>> - Move documentation to the property so the setters and getters inherit it, with an exception for the special case of JobSettings.setPageRanges()
>> - Override toString() on the properties in JobSettings so it doesn't delegate to the JobSettings class.
>> - Add a manual test program just so you can see what toString() does. No pass or fail, just informative.
>> 
>> This will need a CSR but I won't create that until the review is done.
>
> Phil Race has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8269638: Property methods, setters, and getters in printing API should be final

modules/javafx.graphics/src/main/java/javafx/print/PrinterJob.java line 212:

> 210: 
> 211:     /**
> 212:      * Property representing the {@code Printer} for this job.

I guess we normally used <code></code> throughout . Probably better to be consistent.

tests/manual/printing/JobSettingsInfo.java line 31:

> 29: import javafx.collections.FXCollections;
> 30: import javafx.collections.ObservableList;
> 31: import javafx.print.*;

Maybe we can do away with this wildcard and import explicit class.. Also, down below one more place...

-------------

PR: https://git.openjdk.java.net/jfx/pull/574