RFR: (jaxp) 8003260 : some fields should be package protected

Joe Wang huizhe.wang at oracle.com
Fri Dec 14 19:49:04 UTC 2012



On 12/14/2012 10:36 AM, Lance Andersen - Oracle wrote:
> Hi Joe,
>
> Shouldn't this also be private:
>
>   static final char [] xmlDecl = {'<','?','x','m','l'};

A subclass in the same package, XMLDocumentScannerImpl, referred to it. 
That's why I made it package private.

Joe
>
> otherwise it is fine
> Best
> Lance
> On Dec 14, 2012, at 1:33 PM, Joe Wang wrote:
>
>> Hi,
>>
>> This is one of the three [findbug] issues. I've checked with Drew. None of them are vulnerabilities. Nonetheless, the fields should have been private or package private.
>>
>> webrev:
>> http://cr.openjdk.java.net/~joehw/7u12/8003260/webrev/
>>
>> Test:
>> No new test. Existing regression tests passed.
>>
>> Thanks,
>> Joe
>
>
> Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
> Oracle Java Engineering
> 1 Network Drive
> Burlington, MA 01803
> Lance.Andersen at oracle.com
>



More information about the core-libs-dev mailing list