hg: jdk7/tl/langtools: 7003550: Loosen modeling requirements for annotation processing erroneous code

Joe Darcy joe.darcy at oracle.com
Thu Dec 9 20:03:02 PST 2010


brucechapman at paradise.net.nz wrote:
> Quoting Joe Darcy <joe.darcy at oracle.com>:
>
>   
>> On 12/9/2010 1:45 AM, Bruce Chapman wrote:
>>     
>>> On 9/12/2010 6:17 p.m., joe.darcy at oracle.com wrote:
>>>       
>>>> Changeset: 23fc45d3a572
>>>> Author: darcy
>>>> Date: 2010-12-08 21:21 -0800
>>>> URL: http://hg.openjdk.java.net/jdk7/tl/langtools/rev/23fc45d3a572
>>>>
>>>> 7003550: Loosen modeling requirements for annotation processing 
>>>> erroneous code
>>>> Reviewed-by: jjg
>>>>
>>>> ! src/share/classes/javax/lang/model/element/package-info.java
>>>>
>>>>
>>>>         
>  
>   
>>> + * code is not syntactically well-formed or has some other
>>> + * irrecoverable error that could not be removed by the generation
>>>       
>> of
>>     
>>> + * new types, a model may or may not be provided as a quality of
>>> + * implementation issue.
>>> + * If a program is syntactically valid but erroneous in some other
>>> + * fashion, any returned model must have no less information than if
>>> + * all the method bodies in the program were replaced by {@code
>>>       
>> "throw
>>     
>>> + * new RuntimeException();"}. If a program refers to a missing type 
>>> XYZ, A wee nit. Either the sentence starting at "If a program is 
>>> syntactically.." is part of the preceding paragraph in which case it 
>>> ought not start on a new line, OR it is the start of a new paragraph 
>>> so should be preceded by </p><p> - at present it looks like a new 
>>> paragraph in source code, but won't in the javadoc. Bruce
>>>
>>>       
>> Hi Bruce.
>>
>> That was actually intentional to avoid cascading formatting changes. In
>>
>> newer code, I've taken to sometimes separating the sentences of each 
>> paragraph with whitespace to avoid such reformatting issues down the
>> line.
>>     
>
> Ok,  that's a good idea, for consistency did you want to start 'If a program
> refers to a missing type..' on a new line also?
>
>   

If there is a more systematic pass over the JSR 269 API, I'll make this 
change universally, but for now I primarily just wanted to get in the 
spec change.

-Joe




More information about the compiler-dev mailing list