Mercurial > hg > openjdk > jigsaw > jaxp
changeset 406:b1fdb101c82e
8003260: [findbug] some fields should be package protected
Summary: change public or protected mutable static fields to private or package private.
Reviewed-by: lancea
author | joehw |
---|---|
date | Fri, 14 Dec 2012 13:24:34 -0800 |
parents | 83df3493ca3c |
children | 8a20e948b806 |
files | src/com/sun/org/apache/xerces/internal/impl/XMLDocumentFragmentScannerImpl.java src/com/sun/org/apache/xerces/internal/impl/XMLEntityScanner.java |
diffstat | 2 files changed, 7 insertions(+), 4 deletions(-) [+] |
line wrap: on
line diff
--- a/src/com/sun/org/apache/xerces/internal/impl/XMLDocumentFragmentScannerImpl.java Thu Nov 29 11:30:24 2012 -0800 +++ b/src/com/sun/org/apache/xerces/internal/impl/XMLDocumentFragmentScannerImpl.java Fri Dec 14 13:24:34 2012 -0800 @@ -193,9 +193,12 @@ null, }; - protected static final char [] cdata = {'[','C','D','A','T','A','['}; - protected static final char [] xmlDecl = {'<','?','x','m','l'}; - protected static final char [] endTag = {'<','/'}; + private static final char [] cdata = {'[','C','D','A','T','A','['}; + private static final char [] endTag = {'<','/'}; + + //this variable is also used by XMLDocumentScannerImpl in the same package + static final char [] xmlDecl = {'<','?','x','m','l'}; + // debugging /** Debug scanner state. */
--- a/src/com/sun/org/apache/xerces/internal/impl/XMLEntityScanner.java Thu Nov 29 11:30:24 2012 -0800 +++ b/src/com/sun/org/apache/xerces/internal/impl/XMLEntityScanner.java Fri Dec 14 13:24:34 2012 -0800 @@ -71,7 +71,7 @@ /** Listeners which should know when load is being called */ private Vector listeners = new Vector(); - public static final boolean [] VALID_NAMES = new boolean[127]; + private static final boolean [] VALID_NAMES = new boolean[127]; /** * Debug printing of buffer. This debugging flag works best when you