Mercurial > hg > release > icedtea7-forest-2.6 > jaxp
changeset 519:f7a0897308d5
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 | Mon, 17 Dec 2012 22:54:38 -0800 |
parents | 3db0cfb50777 |
children | 900662381e92 |
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 Wed Dec 05 16:04:10 2012 -0800 +++ b/src/com/sun/org/apache/xerces/internal/impl/XMLDocumentFragmentScannerImpl.java Mon Dec 17 22:54:38 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 Wed Dec 05 16:04:10 2012 -0800 +++ b/src/com/sun/org/apache/xerces/internal/impl/XMLEntityScanner.java Mon Dec 17 22:54:38 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