<Swing Dev> [OpenJDK 2D-Dev] [10] Review Request: 8187399 Different problems in the javadoc's links in java.desktop package
Sergey Bylokhov
Sergey.Bylokhov at oracle.com
Sat Sep 16 01:46:48 UTC 2017
That's of course a stylistic issue, but I think that the keywords(not
necessary a java) look better when they are highlighted. Depending from
the style it can look like this:
http://cr.openjdk.java.net/~serb/8187399/img.png
Here is an updated webrev:
http://cr.openjdk.java.net/~serb/8187399/webrev.01
- The link to the "Apple Developer Documentation" was added.
- Info.plist is also wrapped in the {@code}
- Mac OS was replaced by macOS
On 9/15/17 12:34, Phil Race wrote:
> 771 * @implNote Please note that for Mac OS, notifications
> 772 * are only sent if the Java app is a bundled application,
> 773 * with a {@code CFBundleDocumentTypes} array present in its
> 774 * Info.plist.
>
>
> Should we be using {@code ..} for CFBundleDocumentTypes. It is not Java
> code,
> but this makes it look like it is.
>
> I think we should have some reference to "go see Apple developer docs"
> to find out what
> that and Info.plist are .. even if it not a direct link.
>
> -phil.
>
>
> On 09/13/2017 03:39 PM, Sergey Bylokhov wrote:
>> Hello,
>> Please review the fix for jdk10.
>>
>> Some of the tidy warnings were fixed.
>> Report is here:
>> http://cr.openjdk.java.net/~jjg/doc-report/jdk-by-module/java.desktop/report.html
>>
>>
>> Description:
>> - java/awt/Desktop.java: the links to "developer.apple.com" were
>> removed, it seems that the URLs for these documents are changed over
>> time as well as content. We will mention only "CFBundleDocumentTypes"
>> and "Info.plist" in the description.
>> - javax/print/attribute/standard/PresentationDirection.java: the link
>> was changed from ftp://ftp.pwg.org/pub/pwg/standards/pwg5100.3.pdf
>> to ftp://ftp.pwg.org/pub/pwg/standards/temp_archive/pwg5100.3.pdf ,
>> but I guess this is a stable location because the files are there from
>> 2004.
>> - others are just broken links which were fixed.
>>
>> I checked the whole list of links in the desktop module, and found
>> that some of them are broken. For example I cannot find an official
>> place for the tiff specification(and related documents). Also during
>> migration from sun.com we lost some documents, I leave such links
>> as-is, and plan to resurrect the documents somewhere.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8187399
>> Webrev can be found at:
>> http://cr.openjdk.java.net/~serb/8187399/webrev.00
>>
>>
>
--
Best regards, Sergey.
More information about the swing-dev
mailing list