RFR: 8298050: Add links to graph output for javadoc
Pavel Rappo
prappo at openjdk.org
Thu Dec 8 16:48:34 UTC 2022
On Thu, 8 Dec 2022 16:22:30 GMT, Jonathan Gibbons <jjg at openjdk.org> wrote:
>> make/jdk/src/classes/build/tools/taglet/SealedGraph.java line 207:
>>
>>> 205: private static String relativeLink(TypeElement rootNode, TypeElement node) {
>>> 206: var backNavigator = rootNode.getQualifiedName().toString().chars()
>>> 207: .filter(c -> '.' == c)
>>
>> "Yoda conditions" are usually frowned upon. Can `c` be `null`?
>
> `c` is a character
`c` is an `int`, but even that does not matter. I read off `==` but somehow pictured `equals` in my head; so, null-ness does not matter here at all. That said, unnecessary "Yoda conditions" should probably be avoided.
-------------
PR: https://git.openjdk.org/jdk/pull/11580
More information about the build-dev
mailing list