On Mon, 10 Aug 2026 22:22:32 GMT, Andy Goryachev <[email protected]> wrote:

>> Alexander Matveev has updated the pull request incrementally with one 
>> additional commit since the last revision:
>> 
>>   8294872: Document "resource" protocol [v2]
>
> modules/javafx.media/src/main/java/javafx/scene/media/package-info.java line 
> 118:
> 
>> 116:  *
>> 117:  * <table border="1">
>> 118:  * <caption>Supported Schemes / Protocols Table</caption>
> 
> L119: the header says "Protocol" while the table lists schemes and protocols.
> 
> I understand there is some mixing of related things - for example, the "P" in 
> HTTP is for "protocol", but "http" (or "http:") is a URI scheme.
> 
> So maybe revert adding "Schemes /" in L115,118 and just keep saying 
> "protocol"?
> What do you think?

Agree. I will change it to "Supported URL Protocols".

-------------

PR Review Comment: https://git.openjdk.org/jfx/pull/2251#discussion_r3867375651

Reply via email to