cvs commit: jakarta-commons/digester/src/java/org/apache/commons/digester Digester.java

[craigmcc]

craigmcc    02/01/26 17:40:42

  Modified:    digester/src/java/org/apache/commons/digester Digester.java
  Log:
  Consistently log peek() and pop() violations that run into an
  EmptyStackException at log level "warn" instead of either using "info" or not
  logging, as was the previous situation.
  
  Revision  Changes    Path
  1.43      +10 -7     
jakarta-commons/digester/src/java/org/apache/commons/digester/Digester.java
  
  Index: Digester.java
  ===================================================================
  RCS file: 
/home/cvs/jakarta-commons/digester/src/java/org/apache/commons/digester/Digester.java,v
  retrieving revision 1.42
  retrieving revision 1.43
  diff -u -r1.42 -r1.43
  --- Digester.java     23 Jan 2002 21:25:22 -0000      1.42
  +++ Digester.java     27 Jan 2002 01:40:42 -0000      1.43
  @@ -1,7 +1,7 @@
   /*
  - * $Header: 
/home/cvs/jakarta-commons/digester/src/java/org/apache/commons/digester/Digester.java,v
 1.42 2002/01/23 21:25:22 sanders Exp $
  - * $Revision: 1.42 $
  - * $Date: 2002/01/23 21:25:22 $
  + * $Header: 
/home/cvs/jakarta-commons/digester/src/java/org/apache/commons/digester/Digester.java,v
 1.43 2002/01/27 01:40:42 craigmcc Exp $
  + * $Revision: 1.43 $
  + * $Date: 2002/01/27 01:40:42 $
    *
    * ====================================================================
    *
  @@ -112,7 +112,7 @@
    *
    * @author Craig McClanahan
    * @author Scott Sanders
  - * @version $Revision: 1.42 $ $Date: 2002/01/23 21:25:22 $
  + * @version $Revision: 1.43 $ $Date: 2002/01/27 01:40:42 $
    */
   
   public class Digester extends DefaultHandler {
  @@ -1763,7 +1763,7 @@
           try {
               return (stack.peek());
           } catch (EmptyStackException e) {
  -            log.info("Empty stack (returning null)");
  +            log.warn("Empty stack (returning null)");
               return (null);
           }
   
  @@ -1783,7 +1783,7 @@
           try {
               return (stack.peek(n));
           } catch (EmptyStackException e) {
  -            log.info("Empty stack (returning null)");
  +            log.warn("Empty stack (returning null)");
               return (null);
           }
   
  @@ -1799,7 +1799,7 @@
           try {
               return (stack.pop());
           } catch (EmptyStackException e) {
  -            log.info("Empty stack (returning null)");
  +            log.warn("Empty stack (returning null)");
               return (null);
           }
   
  @@ -1890,6 +1890,7 @@
           try {
               return (params.peek());
           } catch (EmptyStackException e) {
  +            log.warn("Empty stack (returning null)");
               return (null);
           }
   
  @@ -1909,6 +1910,7 @@
           try {
               return (params.peek(n));
           } catch (EmptyStackException e) {
  +            log.warn("Empty stack (returning null)");
               return (null);
           }
   
  @@ -1924,6 +1926,7 @@
           try {
               return (params.pop());
           } catch (EmptyStackException e) {
  +            log.warn("Empty stack (returning null)");
               return (null);
           }
   
  
  
  

--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>