Author: violetagg Date: Wed Sep 14 06:37:13 2016 New Revision: 1760640 URL: http://svn.apache.org/viewvc?rev=1760640&view=rev Log: Format the code. No functional changes.
Modified: tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java Modified: tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java URL: http://svn.apache.org/viewvc/tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java?rev=1760640&r1=1760639&r2=1760640&view=diff ============================================================================== --- tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java (original) +++ tomcat/trunk/java/org/apache/tomcat/util/digester/Digester.java Wed Sep 14 06:37:13 2016 @@ -78,10 +78,9 @@ public class Digester extends DefaultHan // ---------------------------------------------------------- Static Fields - private static class SystemPropertySource - implements IntrospectionUtils.PropertySource { + private static class SystemPropertySource implements IntrospectionUtils.PropertySource { @Override - public String getProperty( String key ) { + public String getProperty(String key) { ClassLoader cl = Thread.currentThread().getContextClassLoader(); if (cl instanceof PermissionCheck) { Permission p = new PropertyPermission(key, "read"); @@ -93,29 +92,32 @@ public class Digester extends DefaultHan } } - protected static IntrospectionUtils.PropertySource source[] = - new IntrospectionUtils.PropertySource[] { new SystemPropertySource() }; + protected static IntrospectionUtils.PropertySource source[] = new IntrospectionUtils.PropertySource[] { + new SystemPropertySource() }; static { String className = System.getProperty("org.apache.tomcat.util.digester.PROPERTY_SOURCE"); - if (className!=null) { + if (className != null) { IntrospectionUtils.PropertySource[] sources = new IntrospectionUtils.PropertySource[2]; sources[1] = source[0]; - ClassLoader[] cls = new ClassLoader[] {Digester.class.getClassLoader(),Thread.currentThread().getContextClassLoader()}; + ClassLoader[] cls = new ClassLoader[] { Digester.class.getClassLoader(), + Thread.currentThread().getContextClassLoader() }; boolean initialized = false; - for (int i=0; i<cls.length && (!initialized); i++) { + for (int i = 0; i < cls.length && (!initialized); i++) { try { - Class<?> clazz = Class.forName(className,true,cls[i]); - IntrospectionUtils.PropertySource src = (IntrospectionUtils.PropertySource)clazz.newInstance(); + Class<?> clazz = Class.forName(className, true, cls[i]); + IntrospectionUtils.PropertySource src = (IntrospectionUtils.PropertySource) clazz + .newInstance(); sources[0] = src; initialized = true; } catch (Throwable t) { ExceptionUtils.handleThrowable(t); - LogFactory.getLog("org.apache.tomcat.util.digester.Digester"). - error("Unable to load property source["+className+"].",t); + LogFactory.getLog("org.apache.tomcat.util.digester.Digester") + .error("Unable to load property source[" + className + "].", t); } } - if (initialized) source = sources; + if (initialized) + source = sources; } } @@ -171,7 +173,7 @@ public class Digester extends DefaultHan * The URLs of entityValidator that have been registered, keyed by the public * identifier that corresponds. */ - protected HashMap<String,String> entityValidator = new HashMap<>(); + protected HashMap<String, String> entityValidator = new HashMap<>(); /** @@ -212,7 +214,7 @@ public class Digester extends DefaultHan * is required because documents can declare nested uses of the same * prefix for different Namespace URIs). */ - protected HashMap<String,ArrayStack<String>> namespaces = new HashMap<>(); + protected HashMap<String, ArrayStack<String>> namespaces = new HashMap<>(); /** @@ -289,15 +291,13 @@ public class Digester extends DefaultHan /** * The Log to which most logging calls will be made. */ - protected Log log = - LogFactory.getLog("org.apache.tomcat.util.digester.Digester"); + protected Log log = LogFactory.getLog("org.apache.tomcat.util.digester.Digester"); /** * The Log to which all SAX event related logging calls will be made. */ - protected Log saxLog = - LogFactory.getLog("org.apache.tomcat.util.digester.Digester.sax"); + protected Log saxLog = LogFactory.getLog("org.apache.tomcat.util.digester.Digester.sax"); // ------------------------------------------------------------- Properties @@ -342,8 +342,7 @@ public class Digester extends DefaultHan return (this.classLoader); } if (this.useContextClassLoader) { - ClassLoader classLoader = - Thread.currentThread().getContextClassLoader(); + ClassLoader classLoader = Thread.currentThread().getContextClassLoader(); if (classLoader != null) { return (classLoader); } @@ -421,9 +420,8 @@ public class Digester extends DefaultHan * @throws SAXNotSupportedException Error creating parser * @throws SAXNotRecognizedException Error creating parser */ - public SAXParserFactory getFactory() - throws SAXNotRecognizedException, SAXNotSupportedException, - ParserConfigurationException { + public SAXParserFactory getFactory() throws SAXNotRecognizedException, SAXNotSupportedException, + ParserConfigurationException { if (factory == null) { factory = SAXParserFactory.newInstance(); @@ -431,21 +429,15 @@ public class Digester extends DefaultHan factory.setNamespaceAware(namespaceAware); // Preserve xmlns attributes if (namespaceAware) { - factory.setFeature( - "http://xml.org/sax/features/namespace-prefixes", - true); + factory.setFeature("http://xml.org/sax/features/namespace-prefixes", true); } factory.setValidating(validating); if (validating) { // Enable DTD validation - factory.setFeature( - "http://xml.org/sax/features/validation", - true); + factory.setFeature("http://xml.org/sax/features/validation", true); // Enable schema validation - factory.setFeature( - "http://apache.org/xml/features/validation/schema", - true); + factory.setFeature("http://apache.org/xml/features/validation/schema", true); } } return (factory); @@ -473,9 +465,8 @@ public class Digester extends DefaultHan * @exception SAXNotSupportedException if the property name is * recognized but not supported */ - public void setFeature(String feature, boolean value) - throws ParserConfigurationException, SAXNotRecognizedException, - SAXNotSupportedException { + public void setFeature(String feature, boolean value) throws ParserConfigurationException, + SAXNotRecognizedException, SAXNotSupportedException { getFactory().setFeature(feature, value); @@ -563,7 +554,7 @@ public class Digester extends DefaultHan * Set the public id of the current file being parse. * @param publicId the DTD/Schema public's id. */ - public void setPublicId(String publicId){ + public void setPublicId(String publicId) { this.publicId = publicId; } @@ -644,7 +635,7 @@ public class Digester extends DefaultHan * recognized but not supported */ public Object getProperty(String property) - throws SAXNotRecognizedException, SAXNotSupportedException { + throws SAXNotRecognizedException, SAXNotSupportedException { return (getParser().getProperty(property)); @@ -809,21 +800,20 @@ public class Digester extends DefaultHan * @exception SAXException if no XMLReader can be instantiated */ public XMLReader getXMLReader() throws SAXException { - if (reader == null){ + if (reader == null) { reader = getParser().getXMLReader(); } reader.setDTDHandler(this); reader.setContentHandler(this); - if (entityResolver == null){ + if (entityResolver == null) { reader.setEntityResolver(this); } else { reader.setEntityResolver(entityResolver); } - reader.setProperty( - "http://xml.org/sax/properties/lexical-handler", this); + reader.setProperty("http://xml.org/sax/properties/lexical-handler", this); reader.setErrorHandler(this); return reader; @@ -843,8 +833,7 @@ public class Digester extends DefaultHan * @exception SAXException if a parsing error is to be reported */ @Override - public void characters(char buffer[], int start, int length) - throws SAXException { + public void characters(char buffer[], int start, int length) throws SAXException { if (saxLog.isDebugEnabled()) { saxLog.debug("characters(" + new String(buffer, start, length) + ")"); @@ -865,8 +854,7 @@ public class Digester extends DefaultHan if (saxLog.isDebugEnabled()) { if (getCount() > 1) { - saxLog.debug("endDocument(): " + getCount() + - " elements left"); + saxLog.debug("endDocument(): " + getCount() + " elements left"); } else { saxLog.debug("endDocument()"); } @@ -910,15 +898,14 @@ public class Digester extends DefaultHan * @exception SAXException if a parsing error is to be reported */ @Override - public void endElement(String namespaceURI, String localName, - String qName) throws SAXException { + public void endElement(String namespaceURI, String localName, String qName) + throws SAXException { boolean debug = log.isDebugEnabled(); if (debug) { if (saxLog.isDebugEnabled()) { - saxLog.debug("endElement(" + namespaceURI + "," + localName + - "," + qName + ")"); + saxLog.debug("endElement(" + namespaceURI + "," + localName + "," + qName + ")"); } log.debug(" match='" + match + "'"); log.debug(" bodyText='" + bodyText + "'"); @@ -1037,12 +1024,10 @@ public class Digester extends DefaultHan * @exception SAXException if a parsing error is to be reported */ @Override - public void ignorableWhitespace(char buffer[], int start, int len) - throws SAXException { + public void ignorableWhitespace(char buffer[], int start, int len) throws SAXException { if (saxLog.isDebugEnabled()) { - saxLog.debug("ignorableWhitespace(" + - new String(buffer, start, len) + ")"); + saxLog.debug("ignorableWhitespace(" + new String(buffer, start, len) + ")"); } // No processing required @@ -1059,8 +1044,7 @@ public class Digester extends DefaultHan * @exception SAXException if a parsing error is to be reported */ @Override - public void processingInstruction(String target, String data) - throws SAXException { + public void processingInstruction(String target, String data) throws SAXException { if (saxLog.isDebugEnabled()) { saxLog.debug("processingInstruction('" + target + "','" + data + "')"); @@ -1155,14 +1139,12 @@ public class Digester extends DefaultHan * @exception SAXException if a parsing error is to be reported */ @Override - public void startElement(String namespaceURI, String localName, - String qName, Attributes list) + public void startElement(String namespaceURI, String localName, String qName, Attributes list) throws SAXException { boolean debug = log.isDebugEnabled(); if (saxLog.isDebugEnabled()) { - saxLog.debug("startElement(" + namespaceURI + "," + localName + "," + - qName + ")"); + saxLog.debug("startElement(" + namespaceURI + "," + localName + "," + qName + ")"); } // Parse system properties @@ -1227,8 +1209,7 @@ public class Digester extends DefaultHan * @exception SAXException if a parsing error is to be reported */ @Override - public void startPrefixMapping(String prefix, String namespaceURI) - throws SAXException { + public void startPrefixMapping(String prefix, String namespaceURI) throws SAXException { if (saxLog.isDebugEnabled()) { saxLog.debug("startPrefixMapping(" + prefix + "," + namespaceURI + ")"); @@ -1259,8 +1240,7 @@ public class Digester extends DefaultHan public void notationDecl(String name, String publicId, String systemId) { if (saxLog.isDebugEnabled()) { - saxLog.debug("notationDecl(" + name + "," + publicId + "," + - systemId + ")"); + saxLog.debug("notationDecl(" + name + "," + publicId + "," + systemId + ")"); } } @@ -1275,12 +1255,11 @@ public class Digester extends DefaultHan * @param notation The name of the associated notation */ @Override - public void unparsedEntityDecl(String name, String publicId, - String systemId, String notation) { + public void unparsedEntityDecl(String name, String publicId, String systemId, String notation) { if (saxLog.isDebugEnabled()) { - saxLog.debug("unparsedEntityDecl(" + name + "," + publicId + "," + - systemId + "," + notation + ")"); + saxLog.debug("unparsedEntityDecl(" + name + "," + publicId + "," + systemId + "," + + notation + ")"); } } @@ -1294,7 +1273,7 @@ public class Digester extends DefaultHan * This must be called before the first call to <code>parse()</code>. * @param entityResolver a class that implement the <code>EntityResolver</code> interface. */ - public void setEntityResolver(EntityResolver entityResolver){ + public void setEntityResolver(EntityResolver entityResolver) { this.entityResolver = entityResolver; } @@ -1303,17 +1282,17 @@ public class Digester extends DefaultHan * Return the Entity Resolver used by the SAX parser. * @return Return the Entity Resolver used by the SAX parser. */ - public EntityResolver getEntityResolver(){ + public EntityResolver getEntityResolver() { return entityResolver; } @Override - public InputSource resolveEntity(String name, String publicId, - String baseURI, String systemId) throws SAXException, IOException { + public InputSource resolveEntity(String name, String publicId, String baseURI, String systemId) + throws SAXException, IOException { if (saxLog.isDebugEnabled()) { - saxLog.debug("resolveEntity('" + publicId + "', '" + systemId + - "', '" + baseURI + "')"); + saxLog.debug( + "resolveEntity('" + publicId + "', '" + systemId + "', '" + baseURI + "')"); } // Has this system identifier been registered? @@ -1333,8 +1312,7 @@ public class Digester extends DefaultHan } else { // try to resolve using system ID if (log.isDebugEnabled()) { - log.debug(" Trying to resolve using system ID '" + - systemId + "'"); + log.debug(" Trying to resolve using system ID '" + systemId + "'"); } entityURL = systemId; // resolve systemId against baseURI if it is not absolute @@ -1346,8 +1324,7 @@ public class Digester extends DefaultHan } } catch (URISyntaxException e) { if (log.isDebugEnabled()) { - log.debug("Invalid URI '" + baseURI + "' or '" + - systemId + "'"); + log.debug("Invalid URI '" + baseURI + "' or '" + systemId + "'"); } } } @@ -1370,8 +1347,7 @@ public class Digester extends DefaultHan // ----------------------------------------------- LexicalHandler Methods @Override - public void startDTD(String name, String publicId, String systemId) - throws SAXException { + public void startDTD(String name, String publicId, String systemId) throws SAXException { setPublicId(publicId); } @@ -1389,9 +1365,8 @@ public class Digester extends DefaultHan @Override public void error(SAXParseException exception) throws SAXException { - log.error("Parse Error at line " + exception.getLineNumber() + - " column " + exception.getColumnNumber() + ": " + - exception.getMessage(), exception); + log.error("Parse Error at line " + exception.getLineNumber() + " column " + + exception.getColumnNumber() + ": " + exception.getMessage(), exception); if (errorHandler != null) { errorHandler.error(exception); } @@ -1410,9 +1385,8 @@ public class Digester extends DefaultHan @Override public void fatalError(SAXParseException exception) throws SAXException { - log.error("Parse Fatal Error at line " + exception.getLineNumber() + - " column " + exception.getColumnNumber() + ": " + - exception.getMessage(), exception); + log.error("Parse Fatal Error at line " + exception.getLineNumber() + " column " + + exception.getColumnNumber() + ": " + exception.getMessage(), exception); if (errorHandler != null) { errorHandler.fatalError(exception); } @@ -1430,10 +1404,11 @@ public class Digester extends DefaultHan */ @Override public void warning(SAXParseException exception) throws SAXException { - if (errorHandler != null) { - log.warn("Parse Warning Error at line " + exception.getLineNumber() + - " column " + exception.getColumnNumber() + ": " + - exception.getMessage(), exception); + if (errorHandler != null) { + log.warn( + "Parse Warning Error at line " + exception.getLineNumber() + " column " + + exception.getColumnNumber() + ": " + exception.getMessage(), + exception); errorHandler.warning(exception); } @@ -1461,6 +1436,8 @@ public class Digester extends DefaultHan return (root); } + + /** * Parse the content of the specified input source using this Digester. * Returns the root element from the object stack (if any). @@ -1579,9 +1556,7 @@ public class Digester extends DefaultHan */ public void addCallMethod(String pattern, String methodName) { - addRule( - pattern, - new CallMethodRule(methodName)); + addRule(pattern, new CallMethodRule(methodName)); } @@ -1594,11 +1569,9 @@ public class Digester extends DefaultHan * for a single parameter from the body of this element) * @see CallMethodRule */ - public void addCallMethod(String pattern, String methodName, - int paramCount) { + public void addCallMethod(String pattern, String methodName, int paramCount) { - addRule(pattern, - new CallMethodRule(methodName, paramCount)); + addRule(pattern, new CallMethodRule(methodName, paramCount)); } @@ -1613,8 +1586,7 @@ public class Digester extends DefaultHan */ public void addCallParam(String pattern, int paramIndex) { - addRule(pattern, - new CallParamRule(paramIndex)); + addRule(pattern, new CallParamRule(paramIndex)); } @@ -1629,13 +1601,11 @@ public class Digester extends DefaultHan * object creation will be ignored. * @see FactoryCreateRule */ - public void addFactoryCreate(String pattern, - ObjectCreationFactory creationFactory, - boolean ignoreCreateExceptions) { + public void addFactoryCreate(String pattern, ObjectCreationFactory creationFactory, + boolean ignoreCreateExceptions) { creationFactory.setDigester(this); - addRule(pattern, - new FactoryCreateRule(creationFactory, ignoreCreateExceptions)); + addRule(pattern, new FactoryCreateRule(creationFactory, ignoreCreateExceptions)); } @@ -1648,8 +1618,7 @@ public class Digester extends DefaultHan */ public void addObjectCreate(String pattern, String className) { - addRule(pattern, - new ObjectCreateRule(className)); + addRule(pattern, new ObjectCreateRule(className)); } @@ -1663,11 +1632,9 @@ public class Digester extends DefaultHan * the default Java class name to be created * @see ObjectCreateRule */ - public void addObjectCreate(String pattern, String className, - String attributeName) { + public void addObjectCreate(String pattern, String className, String attributeName) { - addRule(pattern, - new ObjectCreateRule(className, attributeName)); + addRule(pattern, new ObjectCreateRule(className, attributeName)); } @@ -1683,11 +1650,9 @@ public class Digester extends DefaultHan * for a <code>boolean</code> parameter) * @see SetNextRule */ - public void addSetNext(String pattern, String methodName, - String paramType) { + public void addSetNext(String pattern, String methodName, String paramType) { - addRule(pattern, - new SetNextRule(methodName, paramType)); + addRule(pattern, new SetNextRule(methodName, paramType)); } @@ -1700,8 +1665,7 @@ public class Digester extends DefaultHan */ public void addSetProperties(String pattern) { - addRule(pattern, - new SetPropertiesRule()); + addRule(pattern, new SetPropertiesRule()); } @@ -1919,8 +1883,7 @@ public class Digester extends DefaultHan * @return the new exception */ public SAXException createSAXException(String message, Exception e) { - if ((e != null) && - (e instanceof InvocationTargetException)) { + if ((e != null) && (e instanceof InvocationTargetException)) { Throwable t = e.getCause(); if (t instanceof ThreadDeath) { throw (ThreadDeath) t; @@ -1933,8 +1896,8 @@ public class Digester extends DefaultHan } } if (locator != null) { - String error = "Error at (" + locator.getLineNumber() + ", " + - locator.getColumnNumber() + ") : " + message; + String error = "Error at (" + locator.getLineNumber() + ", " + locator.getColumnNumber() + + ") : " + message; if (e != null) { return new SAXParseException(error, locator, e); } else { @@ -2001,13 +1964,11 @@ public class Digester extends DefaultHan for (int i = 0; i < nAttributes; ++i) { String value = newAttrs.getValue(i); try { - String newValue = - IntrospectionUtils.replaceProperties(value, null, source); + String newValue = IntrospectionUtils.replaceProperties(value, null, source); if (value != newValue) { newAttrs.setValue(i, newValue); } - } - catch (Exception e) { + } catch (Exception e) { // ignore - let the attribute have its original value } } @@ -2027,11 +1988,11 @@ public class Digester extends DefaultHan String out; try { out = IntrospectionUtils.replaceProperties(in, null, source); - } catch(Exception e) { + } catch (Exception e) { return bodyText; // return unchanged data } - if (out == in) { + if (out == in) { // No substitutions required. Don't waste memory creating // a new buffer return bodyText; --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org