RFR: 8260592: jpackage tests fail when Desktop is not supported
Alexander Matveev
almatvee at openjdk.java.net
Fri Jan 29 01:16:41 UTC 2021
On Thu, 28 Jan 2021 23:41:44 GMT, Alexey Semenyuk <asemenyuk at openjdk.org> wrote:
>> I would prefer to have separate messages for "headless" and "desktop capable". I can merge these, if you insist.
>
> Duplicated messages are not much help in understanding control flow. I'd change the new message or merge conditions. Whatever you prefer.
It is fine with me to have two separate messages for display and desktop, but it is better to move this check to top of function after if (GraphicsEnvironment.isHeadless()). No point to execute other code.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2291
More information about the core-libs-dev
mailing list