RFR: 8288137: The set of available printers is not updated without application restart [v2]

Phil Race prr at openjdk.org
Fri Jul 8 22:27:44 UTC 2022


On Fri, 8 Jul 2022 21:13:37 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Phil Race has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8288137
>
> modules/javafx.graphics/src/main/java/com/sun/prism/j2d/PrismPrintPipeline.java line 133:
> 
>> 131:                     PrintServiceLookup.lookupPrintServices(null, null);
>> 132:             if ((newServices.length != printerSet.size()) ||
>> 133:                     (lastTime + 120000) > System.currentTimeMillis()) {
> 
> I missed this earlier. The test is backwards. It should be `<` not `>` .

fixed

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

PR: https://git.openjdk.org/jfx/pull/817


More information about the openjfx-dev mailing list