RFR [9] 8080880: some docs cleanup for langtools

alexander stepanov alexander.v.stepanov at oracle.com
Fri May 22 10:14:41 UTC 2015


Hello Jonathan,

Thanks.

 > It is not appropriate to do partial bulk update to the "not a 
supported API"

The responding changes were reverted, please see
http://cr.openjdk.java.net/~avstepan/8080880/webrev.01/

Regards,
Alexander

On 21.05.2015 21:56, Jonathan Gibbons wrote:
>
> On 05/21/2015 09:12 AM, alexander stepanov wrote:
>> Hello,
>>
>> Could you please review the following fix
>> http://cr.openjdk.java.net/~avstepan/8080880/webrev.00/
>> for
>> https://bugs.openjdk.java.net/browse/JDK-8080880
>>
>> Just some minor fix for docs, no other code touched.
>>
>> The affected packages should (probably) not be visible in the new 
>> modular system, but nevertheless...
>>
>> Thanks,
>> Alexander
>
> Alexander,
>
> It is not appropriate to do partial bulk update to the "not a 
> supported API"
> comment at the head of the internal files. It is strongly preferred 
> that this
> part of the comment remain consistent across all source files, so that 
> it is
> amenable to bulk updates, should we wish to do so.
>
> I would suggest you separate the otherwise useful work to fix individual
> minor doc issues from any work involving the standard "not a supported
> API" comment at the head of each file.
>
> -- Jon



More information about the compiler-dev mailing list