I re-ran the script on the .00 version of the patch and it seems to have fixed 
the problem:

http://cr.openjdk.java.net/~bpb/8149593/webrev.01/

Please let me know should I have botched anything else. ;-)

Thanks,

Brian

On Feb 10, 2016, at 2:47 PM, Brian Burkhalter <[email protected]> 
wrote:

> Apparently not. It’s a decade old script. I’ll check it out. Thanks for the 
> catch. Perhaps I could just run it N times instead?
> 
> Brian
> 
> On Feb 10, 2016, at 2:33 PM, Phil Race <[email protected]> wrote:
> 
>> Does your transmogrifier work where there is more than one per line ?
>> 
>> I expected both to be changed here :
>> -     * the <code>JPEGTables</code>. Default value is <code>null</code>.
>> +     * the {@code JPEGTables}. Default value is <code>null</code>.
>> 
>> -phil.
>> 
>> On 02/10/2016 01:41 PM, Brian Burkhalter wrote:
>>> Please review at your convenience.
>>> 
>>> Issue:
>>>         https://bugs.openjdk.java.net/browse/JDK-8149593
>>> Patch:
>>>         http://cr.openjdk.java.net/~bpb/8149593/webrev.00/
>>> 
>>> Summary: Transmogrify “<code>foo</code>” to “{@code foo}”. The patch is 
>>> long but the changes are all the same (quotation marks omitted):
>>> 
>>> “<code>" changed to “{@code “.
>>> “</code>” change to “}”
>>> “2015,” changed to “2016,"
>>> 
>>> Both the source and the docs builds succeed, the latter with no errors or 
>>> warning in these affected classes.
>>> 
>>> Thanks,
>>> 
>>> Brian
>> 
> 

Reply via email to