RFR [9] 8132468: docs: replace <tt> tags (obsolete in html5) for java.io, java.lang, java.math

Stuart Marks stuart.marks at oracle.com
Mon Aug 3 21:47:19 UTC 2015


Hi Alexander,

I think Pavel's advice to run specdiff is useful. It's not too difficult to run 
and in this case it provides a useful cross-check for making a large number of 
markup changes. Contact me directly if you need help with specdiff. I've run 
specdiff over your webrev.01 and I've posted the results here:

http://cr.openjdk.java.net/~smarks/reviews/8132468/specdiff.01/overview-summary.html

They show that of all the changes you made, only four are considered significant 
from specdiff's point of view. Of these, two appear to be merely whitespace 
issues, but there are two changes that do appear significant:

     File(URI)     // constructor
     File.toURI()

In both cases the italic 'f' is intended to be a metasyntactic variable, not an 
actual variable that's part of the code. The italic 'f' should be restored. In 
the case of File.toURI() it appears that the resulting code is now malformed, so 
there's clearly an error here.

You and Daniel Fuchs had an exchange about this a bit earlier. I suspect the 
reason for the   that appears between the opening parenthesis and the 
italic 'f' is so that the tail of the 'f' doesn't crash into parenthesis. In 
other words, the   is there merely for typographical purposes. I suspect 
that after you restore the italic 'f' it'll look bad, so you'll want to restore 
the   as well.

s'marks

On 8/3/15 9:03 AM, Alexander Stepanov wrote:
> Hello Pavel
>
>  > Hi Alexander, if I were you I would run specdiff
> Thanks; but sorry, I'm afraid I haven't enough time for the extra experiments
> just now...
>
>  > It's very easy to go the all the way and lose oneself in there :)
> please accept my condolences :)
>
> Regards,
> Alexander
>
> On 8/3/2015 6:33 PM, Pavel Rappo wrote:
>>> On 3 Aug 2015, at 16:07, Alexander Stepanov <alexander.v.stepanov at oracle.com>
>>> wrote:
>>>
>>> Please see the updated webrev:
>>> http://cr.openjdk.java.net/~avstepan/8132468/webrev.01/
>>>
>>> removed wrapping <code></code> around the links (mostly PrintStream.java,
>>> PrintWriter.java, File.java), plus other changes in File.java
>>>
>>> Thanks,
>>> Alexander
>> Hi Alexander, if I were you I would run specdiff against the changes and
>> upload it along with the RFR.
>>
>> P.S. "I've tried it once, it was awful." [1] It's very easy to go the all the
>> way and lose oneself in there :)
>>
>> ------------------------------------------------------------------------------
>> [1]
>> http://mail.openjdk.java.net/pipermail/core-libs-dev/2014-December/030262.html
>>
>> -Pavel
>>
>



More information about the core-libs-dev mailing list