[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-2d-dev
Subject:    Re: [OpenJDK 2D-Dev] [10] Review Request: 8182410, 8183508, 8181289
From:       Sergey Bylokhov <Sergey.Bylokhov () oracle ! com>
Date:       2017-11-22 19:12:07
Message-ID: 7281217d-a5ae-3151-49c7-8c5b2974b426 () oracle ! com
[Download RAW message or body]

If there are no other comments about this review, I'll push the first 
version(.00) of the fix.

On 23/10/2017 14:42, Sergey Bylokhov wrote:
> 
> Hello,
> Please review the fix for.
> 8182410: missing 'title' in 
> api/javax/swing/plaf/synth/doc-files/componentProperties.html
> 8183508: multi_tsc.html should be updated
> 8181289: Invalid HTML 5 in AWT/Swing docs
> 
> Description:
>    - Illegal characters were removed.
>    - Unsupported tags/properties were removed -like <tt>, <center>, font, 
> etc.(except the tags related to tables which I'll fix later).
>    - HTML5 doctype is set for all files.
>    - The <title> is set for all files.
>    - <a name="" is replaced by <a id=""
>    - Copyrights were added to some files.
> 
> Note that I placed a <head> tag before copyright to solve errors like:
> "A charset attribute on a meta element found after the first 1024 bytes. 
> Fatal Error: Changing encoding at this point would need non-streamable 
> behavior"
> 
> specdiff: 
> http://cr.openjdk.java.net/~serb/8181289/specdiff/overview-summary.html
> 
> Bugs:
>         https://bugs.openjdk.java.net/browse/JDK-8182410
>         https://bugs.openjdk.java.net/browse/JDK-8183508
>         https://bugs.openjdk.java.net/browse/JDK-8181289
> 
> Webrev can be found at: http://cr.openjdk.java.net/~serb/8181289/webrev.00
> 


-- 
Best regards, Sergey.
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic