RFR [9] 8077332: tidy warnings from javax/xml
alexander stepanov
alexander.v.stepanov at oracle.com
Thu Apr 16 14:07:02 UTC 2015
I'm sorry, two extra files touched -
http://cr.openjdk.java.net/~avstepan/8077332/webrev.01/jaxws/src/java.activation/share/classes/javax/activation/MailcapCommandMap.java.udiff.html
http://cr.openjdk.java.net/~avstepan/8077332/webrev.01/jaxws/src/java.activation/share/classes/javax/activation/MimetypesFileTypeMap.java.udiff.html
Hopefully that's all for this bug...
Thanks,
Alexander
On 16.04.2015 15:48, alexander stepanov wrote:
> Please note also that a couple of new files were touched:
>
> http://cr.openjdk.java.net/~avstepan/8077332/webrev.01/jaxws/src/java.annotations.common/share/classes/javax/annotation/PostConstruct.java.udiff.html
> <http://cr.openjdk.java.net/%7Eavstepan/8077332/webrev.01/jaxws/src/java.annotations.common/share/classes/javax/annotation/PostConstruct.java.udiff.html>
>
> http://cr.openjdk.java.net/~avstepan/8077332/webrev.01/jaxws/src/java.annotations.common/share/classes/javax/annotation/PreDestroy.java.udiff.html
> <http://cr.openjdk.java.net/%7Eavstepan/8077332/webrev.01/jaxws/src/java.annotations.common/share/classes/javax/annotation/PreDestroy.java.udiff.html>
>
>
> On 15.04.2015 19:12, alexander stepanov wrote:
>> Hello Joe,
>>
>> The copyright changes were reverted.
>>
>> Please review the updated fix:
>> http://cr.openjdk.java.net/~avstepan/8077332/webrev.01/
>>
>> ("<code></code>" replaced with "{@code}", removed unnecessary "</p>",
>> used "@literal" tag).
>>
>> Thanks,
>> Alexander
>>
>>
>> On 13.04.2015 21:19, huizhe wang wrote:
>>>
>>> On 4/13/2015 4:42 AM, Alan Bateman wrote:
>>>> On 13/04/2015 12:22, alexander stepanov wrote:
>>>>> Hello Joe,
>>>>>
>>>>> Thank you for the notes;
>>>>>
>>>>> > Copyright year shall not be changed.
>>>>>
>>>>> That seems to be a bit controversial point; sometimes (while
>>>>> cleaning docs) I was asked to do that, other times - not to do
>>>>> that. Our internal policy seemingly assigns to change the 2nd date
>>>>> every time the sources were touched (but that may be a question of
>>>>> ambiguous interpretation).
>>>>>
>>>>> But of course I can easily revert these changes if you're totally
>>>>> sure it should be done.
>>>>>
>>>> This has always been confusing. Some areas insist on updating the
>>>> copyright dates, others don't. AFAIK, it has always been optional.
>>>> I think the original assumption was that the update_copyright_year
>>>> script (in the top-level repo) be run periodically to do bulk
>>>> updates. Unfortunately that script doesn't seem to be run very
>>>> often now and this strengthens the case to update the dates on a
>>>> continuous basis. I have not come across the argument that html
>>>> tidy tasks that don't change the javadoc should not update the
>>>> copyright date. The general topic probably should move to jdk9-dev
>>>> and get this decided once and documented in the developer guide.
>>>
>>> I think the key question to ask is: is this the code I can claim
>>> Copyright with? To me, format, code style, html tags and other minor
>>> changes, these are not code changes one can claim copyright with.
>>>
>>> The date of a Copyright establishes how far back the claim is made.
>>> In case where the work is substantially revised, a new Copyright
>>> claim is established, which is what the 2nd year is about.
>>>
>>> In this case, esp. for the JAXP API (e.g. javax.xml.datatype), I'd
>>> like to see the years maintained because those are the years the API
>>> was designed and modified. The "tidy warnings" change did not change
>>> the API.
>>>
>>> -Joe
>>>
>>>>
>>>> -Alan
>>>
>>
>
More information about the core-libs-dev
mailing list