On Mon, 17 Oct 2022 07:12:00 GMT, eduardsdv <d...@openjdk.org> wrote:
>> This fixes a race condition between application and 'Print Job Thread' >> threads when printing. >> >> The race condition occurs when application thread calls `endJob()`, which in >> effect sets the `jobDone` flag to true, >> and when the 'Print Job Thread' thread was in the `synchronized` block in >> `waitForNextPage()` at that time. >> The 'Print Job Thread' thread checks `jobDone` flag after exiting the >> `synchronized` block and, if it is true, skips the last page. >> >> In this fix, not only the `jobDone` is checked, but also that there is no >> other page to be printed. >> It was also needed to introduce a new flag 'jobCanceled', to skip the page >> if the printing was canceled by 'cancelJob()'. > > eduardsdv has updated the pull request incrementally with one additional > commit since the last revision: > > 8295324: Fix race condition in junit test It seems that not all errors were fixed with this PR. Yesterday, a blank page was printed again. I will analyze what this can be due to. ------------- PR: https://git.openjdk.org/jfx/pull/916
