svn commit: r745170 - /tuscany/branches/sca-java-1.x/modules/contribution/src/main/java/org/apache/tuscany/sca/contribution/processor/DefaultValidatingXMLInputFactory.java

[rfeng]

Author: rfeng
Date: Tue Feb 17 17:48:22 2009
New Revision: 745170

URL: http://svn.apache.org/viewvc?rev=745170&view=rev
Log:
Use Logger instead e.printStackTrace

Modified:
    
tuscany/branches/sca-java-1.x/modules/contribution/src/main/java/org/apache/tuscany/sca/contribution/processor/DefaultValidatingXMLInputFactory.java

Modified: 
tuscany/branches/sca-java-1.x/modules/contribution/src/main/java/org/apache/tuscany/sca/contribution/processor/DefaultValidatingXMLInputFactory.java
URL: 
http://svn.apache.org/viewvc/tuscany/branches/sca-java-1.x/modules/contribution/src/main/java/org/apache/tuscany/sca/contribution/processor/DefaultValidatingXMLInputFactory.java?rev=745170&r1=745169&r2=745170&view=diff
==============================================================================
--- 
tuscany/branches/sca-java-1.x/modules/contribution/src/main/java/org/apache/tuscany/sca/contribution/processor/DefaultValidatingXMLInputFactory.java
 (original)
+++ 
tuscany/branches/sca-java-1.x/modules/contribution/src/main/java/org/apache/tuscany/sca/contribution/processor/DefaultValidatingXMLInputFactory.java
 Tue Feb 17 17:48:22 2009
@@ -28,6 +28,8 @@
 import java.security.PrivilegedActionException;
 import java.security.PrivilegedExceptionAction;
 import java.util.List;
+import java.util.logging.Level;
+import java.util.logging.Logger;
 
 import javax.xml.XMLConstants;
 import javax.xml.stream.EventFilter;
@@ -58,7 +60,7 @@
  * @version $Rev$ $Date$
  */
 public class DefaultValidatingXMLInputFactory extends 
ValidatingXMLInputFactory {
-    
+    private static final Logger logger = 
Logger.getLogger(DefaultValidatingXMLInputFactory.class.getName());
     private XMLInputFactory inputFactory;
     private ValidationSchemaExtensionPoint schemas;
     private Monitor monitor;
@@ -87,9 +89,11 @@
      */
      private void error(String message, Object model, Exception ex) {
         if (monitor != null) {
-                Problem problem = new ProblemImpl(this.getClass().getName(), 
"contribution-validation-messages", Severity.ERROR, model, message, ex);
-            monitor.problem(problem);
-        }        
+            Problem problem =
+                new ProblemImpl(this.getClass().getName(), 
"contribution-validation-messages", Severity.ERROR, model,
+                                message, ex);
+            monitor.problem(problem);
+        }        
      }
     
     /**
@@ -126,7 +130,7 @@
                         }
                     });
                 } catch (PrivilegedActionException e) {
-                       error("PrivilegedActionException", url, 
(IOException)e.getException());
+                    error("PrivilegedActionException", url, 
(IOException)e.getException());
                     throw (IOException)e.getException();
                 }
                 sources[i] = new StreamSource(urlStream, uri);
@@ -149,14 +153,14 @@
 
         } catch (Error e) {
             // FIXME Log this, some old JDKs don't support XMLSchema validation
-            //e.printStackTrace();
+            logger.log(Level.WARNING, "XML Schema validation is not supported: 
" + e.getMessage());
         } catch (SAXParseException e) {
-               IllegalStateException ie = new IllegalStateException(e);
-               error("IllegalStateException", schemas, ie);
+            IllegalStateException ie = new IllegalStateException(e);
+            error("IllegalStateException", schemas, ie);
             throw ie;
         } catch (Exception e) {
             //FIXME Log this, some old JDKs don't support XMLSchema validation
-            e.printStackTrace();
+            logger.log(Level.WARNING, "XML Schema validation is not supported: 
" + e.getMessage());
         }
     }