Feedback on page "New APIs in Java 19" about java-almanac HOT 5 CLOSED

Hi @AlanBateman thanks for your feedback! I had the same observation but did not find a proper solution yet. Filtering referenced types by their visibility is currently not really supported by my simple diff tool.

Instead of listing all permitted subclasses I could simply give a class a "sealed" tag if it declares at least one permitted subclass.

• Pro: Very concise diff, new sealed classes can be easily spotted
• Con: Change in list if permitted subclasses is not shown

For me this is a pragmatic solution which at least gives a quick overview. WDYT?

from java-almanac.

What about tags [+ sealed] or [- sealed] or [sealed: permits changed].

However I can already see someone asking to see the permits. Maybe opening permits in a modal view might be an option which does not cause trouble with the current list layout. So [+ sealed] and [sealed: permits changed] should be clickable then.

from java-almanac.

@jnehlmeier As of today these pages are statically rendered. Also it is all backed by an API which I need to consider when I do changes.

from java-almanac.

I don't think it's useful to list permitted subclasses that aren't accessible. In the API docs you'll see it includes "(not exhaustive)" when the permitted subclasses includes subclasses that aren't part of the API. If I read your comment correctly then this is not currently feasible with the diff tool. Okay, I was just providing feedback that recent diffs have noise. You mentioned printing "sealed" when the PermittedSubclasses attribute is present and that could be useful.

from java-almanac.

If I read your comment correctly then this is not currently feasible with the diff tool

Exactly. I process class by class and extract API information from it. For now I can't consider other classes (which would require building an index first). But indeed there are other use cases, for example #4.

from java-almanac.