That's an interesting question. We haven't used Optional for such cases
except the return value of Dialog showAndWait. Wrapping the URL in an
Optional is more self-documenting, but perhaps less convenient if you
know that it is non-null (and basically the same if you don't know,
since this doesn't seem like something you would do anything with other
than asking "is it valid").
Another interesting question: how useful is this API? It was on the list
of impl_* method that Jonathan and I felt would be worth making public
API out of, but I don't know how prevalent its use is?
-- Kevin
Tom Schindl wrote:
As i see it the url is optional so wouldn't make sense to return Optional<URL>
to make this more explicit?
Tom
Von meinem iPhone gesendet
Am 10.12.2015 um 21:54 schrieb Jonathan Giles <jonathan.gi...@oracle.com>:
Hi Chien, Kevin,
Please can you review the following JBS issue (and attached patch):
https://bugs.openjdk.java.net/browse/JDK-8145143
As the subject line states, this issue proposes to promote the
Image.impl_getUrl() method to Image.getUrl(), so that it is public API.
--
-- Jonathan