Review request: 8005096: Move a few source files in swing/beaninfo and in a demo.

Sergey Malenkov Sergey.Malenkov at oracle.com
Wed Jan 9 12:03:54 UTC 2013


Hi Fredrik,

Thanks! It looks good to me.

Regards,
SAM

On 09.01.2013 16:07, Fredrik Öhrström wrote:
> 2013/1/9 Sergey Malenkov <Sergey.Malenkov at oracle.com>:
>> First, *BeanInfo classes must be packed to dt.jar, not rt.jar. So, it will
>> be strange to provide sources for nonexistent (by default) classes.
>> Second, *BeanInfo java files are generated automatically. They are not
>> well-formatted and well-documented.
>
> Ok, I have updated the patch, and the beaninfo files are no longer in src.zip.
> Here is the new webrev:
>
> http://cr.openjdk.java.net/~ohrstrom/webrev-8005096-move-a-few-files-v2/
>
> //Fredrik
>



More information about the build-dev mailing list