RFR: 8046817: JDK 8 schemagen tool does not generate xsd files for enum types

[Aleksej Efimov]

Miran,
Thank you for review. I will work with Iaroslav on integration into 
upstream.
Iaroslav,
I can prepare the patch for JAX-B RI source code (as we discussed 
earlier). Do we need to convert new regression test to JAX-B RI unit 
tests OR we can leave it as it is?

Thanks,
Aleksej

On 10/15/2014 03:08 PM, Miroslav Kos wrote:
> Hi Aleksej,
>
> the change looks ok to me, but it would be good to integrate it into 
> upstream (standalone JAX-B RI project) first to be on safe side - 
> there are much more tests then in OpenJDK.
>
> Copying Iaroslav who is JAX-B RI lead and can help you with 
> integration and running tests.
>
> Thanks
> Miran
>
>
> On 14/10/14 09:29, Aleksej Efimov wrote:
>> Hi XML experts,
>> Can I humbly ask  to review this simple fix.
>>
>> Thank you,
>> Aleksej
>>
>> On 08.10.2014 13:09, Aleksej Efimov wrote:
>>> Hello,
>>>
>>> Please, review the fix [1] for the 8046817 [2].
>>> Problem: schemagen tool doesn't generate schema file for the enum 
>>> types [3].
>>> SchemaGenerator class gets a list of annotated elements and filters 
>>> out only the class variables, but annotation parser 
>>> (AnnotationParser.java) filters out the Class and Enum. The proposed 
>>> fix solves the problem: SchemaGenerator filters like AnnotationParser.
>>> Testing: JPRT build and test passes on platforms (with new 
>>> regression test). JTREG tests: javax/xml - no failures.
>>>
>>> Thanks,
>>> Aleksej
>>>
>>> [1] Webrev: http://cr.openjdk.java.net/~aefimov/8046817/9/webrev.00/
>>> [2] Bug: https://bugs.openjdk.java.net/browse/JDK-8046817
>>> [3] Enum type: 
>>> http://cr.openjdk.java.net/~aefimov/8046817/9/webrev.00/jdk/test/javax/xml/ws/8046817/TestEnumType.java.html
>>
>