RFR: 8048021: Remove @version tag in jaxp repo

Henry Jen henry.jen at oracle.com
Tue Jun 24 21:04:45 UTC 2014


Thanks for reviewing, glad you find out where it is.

Cheers,
Henry

On 06/24/2014 01:56 PM, huizhe wang wrote:
> Looks good. Thanks for the extra work!
>
> I think your webrev is here:
> http://cr.openjdk.java.net/~henryjen/jdk9/8048021/0/webrev/
>
> Cheers,
> Joe
>
> On 6/24/2014 1:00 PM, Henry Jen wrote:
>> Hi,
>>
>> As a follow up task, please review the trivial webrev to remove
>> @version tags used with SCM, which is likely to be meaningless to
>> javadoc readers.
>>
>> In this webrev, we only remove @version with $Id or $Revision for
>> .jave or package.html cause appearance to javadoc.
>>
>> We keep those in .properties files also those @version tags with
>> specific version information.
>>
>> Cheers,
>> Henry
>>
>>
>> On 06/20/2014 05:24 PM, Henry Jen wrote:
>>> Looks like it's pretty easy to do a find and replace in files in jaxp
>>> folder. If this is desired, we can do it in a separate webrev to be
>>> clear on changeset history?
>>>
>>> Cheers,
>>> Henry
>>>
>>> On 06/20/2014 03:36 PM, huizhe wang wrote:
>>>> Hi Henry,
>>>>
>>>> Is it possible to add to the @since tool/script to remove repository
>>>> revision notes?  They appear in the API document as if they reflect
>>>> meaningful "version". An example in
>>>> http://cr.openjdk.java.net/~henryjen/jdk9/8047723/0/webrev/src/javax/xml/parsers/DocumentBuilderFactory.java.udiff.html
>>>>
>>>>
>>>>
>>>>
>>>>    * @version $Revision: 1.9 $, $Date: 2010/05/25 16:19:44 $
>>>>
>



More information about the core-libs-dev mailing list