vgritsenko    02/01/21 16:17:13

  Modified:    src/java/org/apache/cocoon Cocoon.java Main.java
               src/java/org/apache/cocoon/acting DatabaseAddAction.java
                        DatabaseDeleteAction.java DatabaseUpdateAction.java
                        OraAddAction.java ScriptAction.java
               src/java/org/apache/cocoon/caching CachedEventObject.java
                        CachedStreamObject.java
               src/java/org/apache/cocoon/components
                        CocoonComponentManager.java
               src/java/org/apache/cocoon/components/language/generator
                        ProgramGeneratorImpl.java
               src/java/org/apache/cocoon/components/language/markup/xsp
                        XSPCookieHelper.java
               src/java/org/apache/cocoon/components/language/markup/xsp/java
                        log.xsl
               src/java/org/apache/cocoon/components/pipeline
                        AbstractStreamPipeline.java
                        CachingEventPipeline.java
                        CachingStreamPipeline.java
               src/java/org/apache/cocoon/components/resolver
                        ResolverImpl.java
               src/java/org/apache/cocoon/components/sax
                        XMLByteStreamInterpreter.java
               src/java/org/apache/cocoon/components/source
                        SourceHandlerImpl.java URLSource.java
               src/java/org/apache/cocoon/components/store
                        FilesystemQueueImpl.java FilesystemStore.java
                        MRUMemoryStore.java StoreJanitorImpl.java
               src/java/org/apache/cocoon/components/url
                        URLFactoryImpl.java
               src/java/org/apache/cocoon/components/xscript
                        XScriptManagerImpl.java
               src/java/org/apache/cocoon/components/xslt
                        XSLTProcessorImpl.java
               src/java/org/apache/cocoon/environment
                        AbstractEnvironment.java
               src/java/org/apache/cocoon/environment/commandline
                        AbstractCommandLineEnvironment.java
                        CommandLineSession.java
                        LinkSamplingEnvironment.java
               src/java/org/apache/cocoon/environment/http HttpContext.java
               src/java/org/apache/cocoon/environment/wrapper
                        EnvironmentWrapper.java RequestParameters.java
                        RequestWrapper.java
               src/java/org/apache/cocoon/i18n XMLResourceBundle.java
               src/java/org/apache/cocoon/reading DatabaseReader.java
               src/java/org/apache/cocoon/servlet CocoonServlet.java
                        ParanoidCocoonServlet.java
               src/java/org/apache/cocoon/sitemap ContentAggregator.java
                        SitemapComponentSelector.java SitemapManager.java
               src/java/org/apache/cocoon/transformation
                        TraxTransformer.java
               src/java/org/apache/cocoon/util/log
                        XMLCocoonLogFormatter.java
  Log:
  Code style (don't use == true)
  
  Revision  Changes    Path
  1.5       +3 -3      xml-cocoon2/src/java/org/apache/cocoon/Cocoon.java
  
  Index: Cocoon.java
  ===================================================================
  RCS file: /home/cvs/xml-cocoon2/src/java/org/apache/cocoon/Cocoon.java,v
  retrieving revision 1.4
  retrieving revision 1.5
  diff -u -r1.4 -r1.5
  --- Cocoon.java       18 Jan 2002 20:13:40 -0000      1.4
  +++ Cocoon.java       22 Jan 2002 00:17:11 -0000      1.5
  @@ -103,7 +103,7 @@
    * @author <a href="mailto:[EMAIL PROTECTED]";>Pierpaolo Fumagalli</a> (Apache 
Software Foundation, Exoffice Technologies)
    * @author <a href="mailto:[EMAIL PROTECTED]";>Stefano Mazzocchi</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Leo Sutic</a>
  - * @version CVS $Revision: 1.4 $ $Date: 2002/01/18 20:13:40 $
  + * @version CVS $Revision: 1.5 $ $Date: 2002/01/22 00:17:11 $
    */
   public class Cocoon
           extends AbstractLoggable
  @@ -366,12 +366,12 @@
           if (getLogger().isDebugEnabled()) {
               getLogger().debug("Configuration version: " + 
conf.getAttribute("version"));
           }
  -        if (Constants.CONF_VERSION.equals(conf.getAttribute("version")) == false) {
  +        if (!Constants.CONF_VERSION.equals(conf.getAttribute("version"))) {
               throw new ConfigurationException("Invalid configuration schema version. 
Must be '" + Constants.CONF_VERSION + "'.");
           }
   
           String userRoles = conf.getAttribute("user-roles", "");
  -        if ("".equals(userRoles) == false) {
  +        if (!"".equals(userRoles)) {
               try {
                   p = (Parser)startupManager.lookup(Parser.ROLE);
                   SAXConfigurationHandler b = new SAXConfigurationHandler();
  
  
  
  1.12      +6 -6      xml-cocoon2/src/java/org/apache/cocoon/Main.java
  
  Index: Main.java
  ===================================================================
  RCS file: /home/cvs/xml-cocoon2/src/java/org/apache/cocoon/Main.java,v
  retrieving revision 1.11
  retrieving revision 1.12
  diff -u -r1.11 -r1.12
  --- Main.java 12 Jan 2002 02:00:10 -0000      1.11
  +++ Main.java 22 Jan 2002 00:17:11 -0000      1.12
  @@ -99,7 +99,7 @@
    * Command line entry point.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Stefano Mazzocchi</a>
  - * @version CVS $Revision: 1.11 $ $Date: 2002/01/12 02:00:10 $
  + * @version CVS $Revision: 1.12 $ $Date: 2002/01/22 00:17:11 $
    */
   public class Main {
   
  @@ -340,7 +340,7 @@
               System.exit(1);
           }
   
  -        if (targets.size() == 0 && xspOnly == false) {
  +        if (targets.size() == 0 && !xspOnly) {
               String error = "Please, specify at least one starting URI.";
               log.error(error);
               System.out.println(error);
  @@ -569,9 +569,9 @@
           ArrayList links = new java.util.ArrayList();
           Iterator i = uris.iterator();
           String next;
  -        while (i.hasNext() == true) {
  +        while (i.hasNext()) {
               next = (String)i.next();
  -            if (links.contains(next) == false) {
  +            if (!links.contains(next)) {
                   links.add(next);
               }
           }
  @@ -584,9 +584,9 @@
                           this.processXSP(url);
                       } else if (this.followLinks) {
                           i = this.processURI(url).iterator();
  -                        while (i.hasNext() == true) {
  +                        while (i.hasNext()) {
                               next = (String)i.next();
  -                            if (links.contains(next) == false) {
  +                            if (!links.contains(next)) {
                                   links.add(next);
                               }
                           }
  
  
  
  1.2       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/acting/DatabaseAddAction.java
  
  Index: DatabaseAddAction.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/acting/DatabaseAddAction.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- DatabaseAddAction.java    3 Jan 2002 12:31:07 -0000       1.1
  +++ DatabaseAddAction.java    22 Jan 2002 00:17:11 -0000      1.2
  @@ -33,7 +33,7 @@
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Berin Loritsch</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Donald Ball</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:07 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:11 $
    */
   public class DatabaseAddAction extends AbstractDatabaseAction implements ThreadSafe 
{
       protected static final Map addStatements = new HashMap();
  @@ -63,7 +63,7 @@
               conn = datasource.getConnection();
               Request request = (Request) objectModel.get(Constants.REQUEST_OBJECT);
   
  -            if (conn.getAutoCommit() == true) {
  +            if (conn.getAutoCommit()) {
                   conn.setAutoCommit(false);
               }
   
  
  
  
  1.2       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/acting/DatabaseDeleteAction.java
  
  Index: DatabaseDeleteAction.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/acting/DatabaseDeleteAction.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- DatabaseDeleteAction.java 3 Jan 2002 12:31:07 -0000       1.1
  +++ DatabaseDeleteAction.java 22 Jan 2002 00:17:11 -0000      1.2
  @@ -32,7 +32,7 @@
    * the keys.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Berin Loritsch</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:07 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:11 $
    */
   public final class DatabaseDeleteAction extends AbstractDatabaseAction implements 
ThreadSafe {
       private static final Map deleteStatements = new HashMap();
  @@ -62,7 +62,7 @@
               conn = datasource.getConnection();
               Request request = (Request) objectModel.get(Constants.REQUEST_OBJECT);
   
  -            if (conn.getAutoCommit() == true) {
  +            if (conn.getAutoCommit()) {
                   conn.setAutoCommit(false);
               }
   
  
  
  
  1.2       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/acting/DatabaseUpdateAction.java
  
  Index: DatabaseUpdateAction.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/acting/DatabaseUpdateAction.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- DatabaseUpdateAction.java 3 Jan 2002 12:31:07 -0000       1.1
  +++ DatabaseUpdateAction.java 22 Jan 2002 00:17:11 -0000      1.2
  @@ -29,7 +29,7 @@
    * only one table at a time to update.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Berin Loritsch</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:07 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:11 $
    */
   public class DatabaseUpdateAction extends AbstractDatabaseAction implements 
ThreadSafe {
       private static final Map updateStatements = new HashMap();
  @@ -59,7 +59,7 @@
               conn = datasource.getConnection();
               Request request = (Request) objectModel.get(Constants.REQUEST_OBJECT);
   
  -            if (conn.getAutoCommit() == true) {
  +            if (conn.getAutoCommit()) {
                   conn.setAutoCommit(false);
               }
   
  
  
  
  1.2       +2 -2      xml-cocoon2/src/java/org/apache/cocoon/acting/OraAddAction.java
  
  Index: OraAddAction.java
  ===================================================================
  RCS file: /home/cvs/xml-cocoon2/src/java/org/apache/cocoon/acting/OraAddAction.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- OraAddAction.java 3 Jan 2002 12:31:07 -0000       1.1
  +++ OraAddAction.java 22 Jan 2002 00:17:11 -0000      1.2
  @@ -35,7 +35,7 @@
    * only one table at a time to update.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Berin Loritsch</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:07 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:11 $
    */
   public class OraAddAction extends DatabaseAddAction {
       private static final Map selectLOBStatements = new HashMap();
  @@ -58,7 +58,7 @@
               conn = datasource.getConnection();
               Request request = (Request) objectModel.get(Constants.REQUEST_OBJECT);
   
  -            if (conn.getAutoCommit() == true) {
  +            if (conn.getAutoCommit()) {
                   conn.setAutoCommit(false);
               }
   
  
  
  
  1.2       +2 -2      xml-cocoon2/src/java/org/apache/cocoon/acting/ScriptAction.java
  
  Index: ScriptAction.java
  ===================================================================
  RCS file: /home/cvs/xml-cocoon2/src/java/org/apache/cocoon/acting/ScriptAction.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- ScriptAction.java 3 Jan 2002 12:31:07 -0000       1.1
  +++ ScriptAction.java 22 Jan 2002 00:17:11 -0000      1.2
  @@ -32,7 +32,7 @@
    * A simple action that executes any script that can be run by the BSF
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Jason Foster</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:07 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:11 $
    */
   
   public class ScriptAction
  @@ -59,7 +59,7 @@
               src = resolver.resolve(scriptName);
               String systemID = src.getSystemId();
   
  -            if (this.getLogger().isDebugEnabled() == true) {
  +            if (this.getLogger().isDebugEnabled()) {
                   getLogger().debug("script source [" + scriptName + "]");
                   getLogger().debug("script resolved to [" + systemID + "]");
               }
  
  
  
  1.2       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/caching/CachedEventObject.java
  
  Index: CachedEventObject.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/caching/CachedEventObject.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- CachedEventObject.java    3 Jan 2002 12:31:08 -0000       1.1
  +++ CachedEventObject.java    22 Jan 2002 00:17:11 -0000      1.2
  @@ -13,7 +13,7 @@
    * This is an cached object as it is stored in the <code>EventCache</code>
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:08 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:11 $
    */
   public final class CachedEventObject implements java.io.Serializable {
   
  @@ -39,7 +39,7 @@
       public boolean isValid(ComponentCacheKey componentKey,
                              CacheValidity     componentValidity) {
           CacheValidity ownValidity = 
(CacheValidity)this.validityObjects.get(componentKey);
  -        if (ownValidity != null && ownValidity.isValid(componentValidity) == true) {
  +        if (ownValidity != null && ownValidity.isValid(componentValidity)) {
               return true;
           }
           return false;
  
  
  
  1.2       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/caching/CachedStreamObject.java
  
  Index: CachedStreamObject.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/caching/CachedStreamObject.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- CachedStreamObject.java   3 Jan 2002 12:31:08 -0000       1.1
  +++ CachedStreamObject.java   22 Jan 2002 00:17:11 -0000      1.2
  @@ -13,7 +13,7 @@
    * This is an cached object as it is stored in the <code>StreamCache</code>
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:08 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:11 $
    */
   public final class CachedStreamObject implements java.io.Serializable {
   
  @@ -39,7 +39,7 @@
       public boolean isValid(ComponentCacheKey componentKey,
                              CacheValidity     componentValidity) {
           CacheValidity ownValidity = 
(CacheValidity)this.validityObjects.get(componentKey);
  -        if (ownValidity != null && ownValidity.isValid(componentValidity) == true) {
  +        if (ownValidity != null && ownValidity.isValid(componentValidity)) {
               return true;
           }
           return false;
  
  
  
  1.2       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/components/CocoonComponentManager.java
  
  Index: CocoonComponentManager.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/CocoonComponentManager.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- CocoonComponentManager.java       7 Jan 2002 12:34:12 -0000       1.1
  +++ CocoonComponentManager.java       22 Jan 2002 00:17:11 -0000      1.2
  @@ -28,7 +28,7 @@
    * by a special lifecycle handling for a RequestLifecycleComponent.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/07 12:34:12 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:11 $
    */
   public final class CocoonComponentManager
   extends ExcaliburComponentManager
  @@ -80,7 +80,7 @@
               final Object[] objects = (Object[])stack.pop();
               final Map components = (Map)objects[2];
               final Iterator iter = components.values().iterator();
  -            while (iter.hasNext() == true) {
  +            while (iter.hasNext()) {
                   final Object[] o = (Object[])iter.next();
                   final Component component = (Component)o[0];
                   ((ComponentManager)o[1]).release( component );
  
  
  
  1.2       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/components/language/generator/ProgramGeneratorImpl.java
  
  Index: ProgramGeneratorImpl.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/language/generator/ProgramGeneratorImpl.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- ProgramGeneratorImpl.java 3 Jan 2002 12:31:09 -0000       1.1
  +++ ProgramGeneratorImpl.java 22 Jan 2002 00:17:11 -0000      1.2
  @@ -39,7 +39,7 @@
   /**
    * The default implementation of <code>ProgramGenerator</code>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Ricardo Rocha</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:09 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:11 $
    */
   public class ProgramGeneratorImpl extends AbstractLoggable
       implements ProgramGenerator, Contextualizable, Composable, Configurable, 
ThreadSafe, Disposable {
  @@ -201,7 +201,7 @@
                       );
               }
       
  -            if (this.autoReload == false) {
  +            if (!this.autoReload) {
                   return programInstance;
               }
       
  
  
  
  1.3       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/components/language/markup/xsp/XSPCookieHelper.java
  
  Index: XSPCookieHelper.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/language/markup/xsp/XSPCookieHelper.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- XSPCookieHelper.java      15 Jan 2002 16:43:20 -0000      1.2
  +++ XSPCookieHelper.java      22 Jan 2002 00:17:11 -0000      1.3
  @@ -43,7 +43,7 @@
    * private static String returnCookieProperty(Map ,String ,int ,String );
    * </pre>
    *
  - * @version CVS $Revision: 1.2 $ $Date: 2002/01/15 16:43:20 $
  + * @version CVS $Revision: 1.3 $ $Date: 2002/01/22 00:17:11 $
    */
   public class XSPCookieHelper extends XSPObjectHelper {
       /**
  @@ -533,7 +533,7 @@
           if (cookies != null && cookies.length > 0) {
               if (retrieveByName) {
                   count = 0;
  -                while (count < cookies.length && matchFound == false) {
  +                while (count < cookies.length && !matchFound) {
                       currentCookie = cookies[count];
   
                       if (currentCookie.getName().equals(cookieName))
  
  
  
  1.2       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/components/language/markup/xsp/java/log.xsl
  
  Index: log.xsl
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/language/markup/xsp/java/log.xsl,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- log.xsl   3 Jan 2002 12:31:11 -0000       1.1
  +++ log.xsl   22 Jan 2002 00:17:11 -0000      1.2
  @@ -11,7 +11,7 @@
   
   <!--
    * @author <a href="mailto:[EMAIL PROTECTED]>Berin Loritsch</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:11 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:11 $
   -->
   
   <!-- XSP Response logicsheet for the Java language -->
  @@ -68,7 +68,7 @@
               String category = <xsl:value-of select="$name"/>;
               org.apache.log.Logger logger = 
org.apache.log.Hierarchy.getDefaultHierarchy().getLoggerFor("cocoon" + 
(("".equals(category))? category : "." + category));
   <!--
  -            if ("".equals(<xsl:value-of select="$filename"/>) == false) {
  +            if (!"".equals(<xsl:value-of select="$filename"/>)) {
                   String file = 
this.avalonContext.get(org.apache.cocoon.Constants.CONTEXT_LOG_DIR) + <xsl:value-of 
select="$filename"/>;
                   org.apache.log.LogTarget[] targets = new org.apache.log.LogTarget[] 
{
                       new org.apache.log.output.FileOutputLogTarget(file)
  
  
  
  1.3       +4 -4      
xml-cocoon2/src/java/org/apache/cocoon/components/pipeline/AbstractStreamPipeline.java
  
  Index: AbstractStreamPipeline.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/pipeline/AbstractStreamPipeline.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- AbstractStreamPipeline.java       15 Jan 2002 11:10:51 -0000      1.2
  +++ AbstractStreamPipeline.java       22 Jan 2002 00:17:11 -0000      1.3
  @@ -30,7 +30,7 @@
    * @author <a href="mailto:[EMAIL PROTECTED]";>Giacomo Pati</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Sylvain Wallez</a>
  - * @version CVS $Revision: 1.2 $ $Date: 2002/01/15 11:10:51 $
  + * @version CVS $Revision: 1.3 $ $Date: 2002/01/22 00:17:11 $
    */
   public abstract class AbstractStreamPipeline extends AbstractLoggable implements 
StreamPipeline, Disposable {
       protected EventPipeline eventPipeline;
  @@ -132,7 +132,7 @@
               connectPipeline();
   
               try {
  -                if (this.serializer.shouldSetContentLength() == true) {
  +                if (this.serializer.shouldSetContentLength()) {
                       // set the output stream
                       ByteArrayOutputStream os = new ByteArrayOutputStream();
                       this.serializer.setOutputStream(os);
  @@ -180,14 +180,14 @@
               // has the read resource been modified?
               long lastModified = this.reader.getLastModified();
               if (lastModified != 0
  -                && environment.isResponseModified(lastModified) == false) {
  +                && !environment.isResponseModified(lastModified)) {
   
                   // environment supports this, so we are finished
                   environment.setResponseIsNotModified();
                   return true;
               }
   
  -            if (this.reader.shouldSetContentLength() == true) {
  +            if (this.reader.shouldSetContentLength()) {
                   ByteArrayOutputStream os = new ByteArrayOutputStream();
                   this.reader.setOutputStream(os);
                   this.reader.generate();
  
  
  
  1.2       +12 -13    
xml-cocoon2/src/java/org/apache/cocoon/components/pipeline/CachingEventPipeline.java
  
  Index: CachingEventPipeline.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/pipeline/CachingEventPipeline.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- CachingEventPipeline.java 3 Jan 2002 12:31:12 -0000       1.1
  +++ CachingEventPipeline.java 22 Jan 2002 00:17:11 -0000      1.2
  @@ -41,7 +41,7 @@
    * does not cache! (If it would cache, the response would be cached twice!)
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:12 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:11 $
    */
   public class CachingEventPipeline
   extends AbstractEventPipeline
  @@ -97,7 +97,7 @@
       public PipelineCacheKey generateKey(Environment environment)
       throws Exception {
           this.setup(environment);
  -        return (this.cacheable == true ? this.pipelineCacheKey : null);
  +        return (this.cacheable ? this.pipelineCacheKey : null);
       }
   
       /**
  @@ -112,7 +112,7 @@
        */
       public Map generateValidity(Environment environment)
       throws Exception {
  -        if (this.cacheable == true) {
  +        if (this.cacheable) {
               return this.validityObjects;
           }
           return null;
  @@ -125,7 +125,7 @@
       public void setStreamPipelineCaches(boolean flag) {
           // if we are cacheable and this method is called
           // reset the pipeline cache key to avoid duplicate caching
  -        if (flag == true) {
  +        if (flag) {
               this.pipelineCacheKey = null;
           }
       }
  @@ -146,18 +146,17 @@
                       Iterator validityIterator = validityObjects.keySet().iterator();
                       ComponentCacheKey validityKey;
                       boolean valid = true;
  -                    while (validityIterator.hasNext() == true && valid == true) {
  +                    while (validityIterator.hasNext() && valid) {
                           validityKey = (ComponentCacheKey)validityIterator.next();
                           valid = cachedObject.isValid(validityKey, 
(CacheValidity)validityObjects.get(validityKey));
  -                        if (getLogger().isDebugEnabled() == true) {
  +                        if (getLogger().isDebugEnabled()) {
                               CacheValidity cachedValidity = 
cachedObject.getCacheValidity(validityKey);
                               getLogger().debug("Compared cached validity '" + 
cachedValidity +
                                   "' with new validity '" + 
validityObjects.get(validityKey) +
  -                                "' : " + (valid == true ? "valid" : "changed"));
  +                                "' : " + (valid ? "valid" : "changed"));
                           }
                       }
  -                    if (valid == true) {
  -
  +                    if (valid) {
                           getLogger().debug("Using valid cached content for '" + 
environment.getURI() + "'.");
                           // get all transformers which are not cacheable
                           int transformerSize = this.transformers.size();
  @@ -228,10 +227,10 @@
        * Cacheable.
        */
       private void setup(Environment environment) throws Exception {
  -        if (this.setupFinished == true) {
  +        if (this.setupFinished) {
               return;
           }
  -        if (this.checkPipeline() == false) {
  +        if (!this.checkPipeline()) {
               throw new ProcessingException("Attempted to process incomplete 
pipeline.");
           }
   
  @@ -266,7 +265,7 @@
                   boolean testTrans = true;
   
                   while (this.firstNotCacheableTransformerIndex < transformerSize
  -                           && testTrans == true) {
  +                           && testTrans) {
                       trans = 
(Transformer)this.transformers.get(this.firstNotCacheableTransformerIndex);
                       if (trans instanceof Cacheable) {
                           transKey = ((Cacheable)trans).generateKey();
  @@ -284,7 +283,7 @@
                       } else {
                           testTrans = false;
                       }
  -                    if (testTrans == true)
  +                    if (testTrans)
                           this.firstNotCacheableTransformerIndex++;
                   }
                   // all transformers are cacheable => pipeline is cacheable
  
  
  
  1.2       +16 -19    
xml-cocoon2/src/java/org/apache/cocoon/components/pipeline/CachingStreamPipeline.java
  
  Index: CachingStreamPipeline.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/pipeline/CachingStreamPipeline.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- CachingStreamPipeline.java        3 Jan 2002 12:31:12 -0000       1.1
  +++ CachingStreamPipeline.java        22 Jan 2002 00:17:11 -0000      1.2
  @@ -41,7 +41,7 @@
    *  </ul>
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:12 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:11 $
    */
   public class CachingStreamPipeline extends AbstractStreamPipeline {
   
  @@ -104,7 +104,7 @@
               // has the read resource been modified?
               long lastModified = this.reader.getLastModified();
               if (lastModified != 0
  -                && environment.isResponseModified(lastModified) == false) {
  +                && !environment.isResponseModified(lastModified)) {
   
                   // environment supports this, so we are finished
                   environment.setResponseIsNotModified();
  @@ -159,17 +159,17 @@
                       Iterator validityIterator = validityObjects.keySet().iterator();
                       ComponentCacheKey validityKey;
                       boolean valid = true;
  -                    while (validityIterator.hasNext() == true && valid == true) {
  +                    while (validityIterator.hasNext() && valid) {
                           validityKey = (ComponentCacheKey)validityIterator.next();
                           valid = cachedObject.isValid(validityKey, 
(CacheValidity)validityObjects.get(validityKey));
  -                        if (getLogger().isDebugEnabled() == true) {
  +                        if (getLogger().isDebugEnabled()) {
                               CacheValidity cachedValidity = 
cachedObject.getCacheValidity(validityKey);
                               getLogger().debug("Compared cached validity '" + 
cachedValidity +
                                   "' with new validity '" + 
validityObjects.get(validityKey) +
  -                                "' : " + (valid == true ? "valid" : "changed"));
  +                                "' : " + (valid ? "valid" : "changed"));
                           }
                       }
  -                    if (valid == true) {
  +                    if (valid) {
                           getLogger().debug("Using valid cached content for '" + 
environment.getURI() + "'.");
   
                           byte[] response = cachedObject.getResponse();
  @@ -180,7 +180,7 @@
                           }
                       }
   
  -                    if(usedCache == false) {
  +                    if(!usedCache) {
                           getLogger().debug("Cached content is invalid for '" + 
environment.getURI() + "'.");
   
                           // remove invalid cached object
  @@ -194,9 +194,9 @@
                   }
               }
   
  -            if (usedCache == false) {
  +            if (!usedCache) {
   
  -                if (this.reader.shouldSetContentLength() == true) {
  +                if (this.reader.shouldSetContentLength()) {
                       ByteArrayOutputStream os = new ByteArrayOutputStream();
                       this.reader.setOutputStream(os);
                       this.reader.generate();
  @@ -299,18 +299,17 @@
                               Iterator validityIterator = 
validityObjects.keySet().iterator();
                               ComponentCacheKey validityKey;
                               boolean valid = true;
  -                            while (validityIterator.hasNext() == true && valid == 
true) {
  +                            while (validityIterator.hasNext() && valid) {
                                   validityKey = 
(ComponentCacheKey)validityIterator.next();
                                   valid = cachedObject.isValid(validityKey, 
(CacheValidity)validityObjects.get(validityKey));
  -                                if (getLogger().isDebugEnabled() == true) {
  +                                if (getLogger().isDebugEnabled()) {
                                       CacheValidity cachedValidity = 
cachedObject.getCacheValidity(validityKey);
                                       getLogger().debug("Compared cached validity '" 
+ cachedValidity +
                                           "' with new validity '" + 
validityObjects.get(validityKey) +
  -                                        "' : " + (valid == true ? "valid" : 
"changed"));
  +                                        "' : " + (valid ? "valid" : "changed"));
                                   }
                               }
  -                            if (valid == true) {
  -
  +                            if (valid) {
                                   getLogger().debug("Using valid cached content for 
'" + environment.getURI() + "'.");
                                   byte[] bytes = cachedObject.getResponse();
                                   if(bytes.length > 0) {
  @@ -320,8 +319,7 @@
                                   }
                               }
   
  -                            if (usedCache == false) {
  -
  +                            if (!usedCache) {
                                   getLogger().debug("Cached content is invalid for '" 
+ environment.getURI() + "'.");
   
                                   // remove invalid cached object
  @@ -338,9 +336,8 @@
                       }
                   }
   
  -                if (usedCache == false) {
  -
  -                    if (this.serializer.shouldSetContentLength() == true) {
  +                if (!usedCache) {
  +                    if (this.serializer.shouldSetContentLength()) {
                           // set the output stream
                           ByteArrayOutputStream os = new ByteArrayOutputStream();
                           this.serializer.setOutputStream(os);
  
  
  
  1.4       +5 -5      
xml-cocoon2/src/java/org/apache/cocoon/components/resolver/ResolverImpl.java
  
  Index: ResolverImpl.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/resolver/ResolverImpl.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- ResolverImpl.java 18 Jan 2002 03:17:23 -0000      1.3
  +++ ResolverImpl.java 22 Jan 2002 00:17:12 -0000      1.4
  @@ -44,7 +44,7 @@
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Davanum Srinivas</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>David Crossley</a>
  - * @version CVS $Revision: 1.3 $ $Date: 2002/01/18 03:17:23 $
  + * @version CVS $Revision: 1.4 $ $Date: 2002/01/22 00:17:12 $
    */
   public class ResolverImpl extends AbstractLoggable
           implements Resolver, Contextualizable, Composable, Parameterizable,
  @@ -79,7 +79,7 @@
           int debugLevel = Debug.getDebug();
           String verbosity = params.getParameter("verbosity", "");
           if (verbosity != "") {
  -            if (this.getLogger().isDebugEnabled() == true) {
  +            if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Setting Catalog resolver "
                   + "verbosity level to " + verbosity);
               }
  @@ -103,7 +103,7 @@
                   catalogURL = 
                       this.context.getResource(catalogFile).toExternalForm();
               }
  -            if (this.getLogger().isDebugEnabled() == true) {
  +            if (this.getLogger().isDebugEnabled()) {
                   this.getLogger().debug("System OASIS Catalog URL is "
                       + catalogURL);
               }
  @@ -116,7 +116,7 @@
           String localCatalogFile = params.getParameter("local-catalog", "");
           if (localCatalogFile != "") {
               try {
  -                if (this.getLogger().isDebugEnabled() == true) {
  +                if (this.getLogger().isDebugEnabled()) {
                       this.getLogger().debug("Additional Catalog is "
                           + localCatalogFile);
                   }
  @@ -176,7 +176,7 @@
           InputSource altInputSource = catalogResolver.resolveEntity(publicId,
                                           systemId);
           if (altInputSource != null) {
  -            if (this.getLogger().isDebugEnabled() == true) {
  +            if (this.getLogger().isDebugEnabled()) {
                   this.getLogger().debug("Resolved catalog entity: "
                       + publicId + " " + altInputSource.getSystemId());
               }
  
  
  
  1.3       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/components/sax/XMLByteStreamInterpreter.java
  
  Index: XMLByteStreamInterpreter.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/sax/XMLByteStreamInterpreter.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- XMLByteStreamInterpreter.java     18 Jan 2002 19:16:48 -0000      1.2
  +++ XMLByteStreamInterpreter.java     22 Jan 2002 00:17:12 -0000      1.3
  @@ -20,7 +20,7 @@
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Stefano Mazzocchi</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version CVS $Revision: 1.2 $ $Date: 2002/01/18 19:16:48 $
  + * @version CVS $Revision: 1.3 $ $Date: 2002/01/22 00:17:12 $
    */
   
   public final class XMLByteStreamInterpreter
  @@ -50,7 +50,7 @@
   
       public void deserialize(Object saxFragment)
       throws SAXException {
  -        if (saxFragment instanceof byte[] == false) {
  +        if (!(saxFragment instanceof byte[])) {
               throw new SAXException("XMLDeserializer needs byte array for 
deserialization.");
           }
           this.input = (byte[])saxFragment;
  
  
  
  1.2       +3 -3      
xml-cocoon2/src/java/org/apache/cocoon/components/source/SourceHandlerImpl.java
  
  Index: SourceHandlerImpl.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/source/SourceHandlerImpl.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- SourceHandlerImpl.java    3 Jan 2002 12:31:14 -0000       1.1
  +++ SourceHandlerImpl.java    22 Jan 2002 00:17:12 -0000      1.2
  @@ -36,7 +36,7 @@
   
   /**
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version $Id: SourceHandlerImpl.java,v 1.1 2002/01/03 12:31:14 giacomo Exp $
  + * @version $Id: SourceHandlerImpl.java,v 1.2 2002/01/22 00:17:12 vgritsenko Exp $
    */
   public final class SourceHandlerImpl
   extends AbstractLoggable
  @@ -67,7 +67,7 @@
               String protocol = null;
               for (int i = 0; i < configs.length; i++) {
                   protocol = configs[i].getAttribute("name");
  -                if (factories.containsKey(protocol) == true) {
  +                if (factories.containsKey(protocol)) {
                       throw new ConfigurationException("SourceFactory defined twice 
for protocol: " + protocol);
                   }
   
  @@ -114,7 +114,7 @@
           }
           Iterator iter = this.sourceFactories.values().iterator();
           SourceFactory current;
  -        while (iter.hasNext() == true) {
  +        while (iter.hasNext()) {
               current = (SourceFactory) iter.next();
               this.deinit(current);
           }
  
  
  
  1.2       +8 -8      
xml-cocoon2/src/java/org/apache/cocoon/components/source/URLSource.java
  
  Index: URLSource.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/source/URLSource.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- URLSource.java    3 Jan 2002 12:31:14 -0000       1.1
  +++ URLSource.java    22 Jan 2002 00:17:12 -0000      1.2
  @@ -32,7 +32,7 @@
    * Description of a source which is described by an URL.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:14 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:12 $
    */
   
   public class URLSource implements ModifiableSource {
  @@ -81,8 +81,8 @@
        * Any exceptions are ignored.
        */
       private void getInfos() {
  -        if (this.gotInfos == false) {
  -            if (this.isFile == true) {
  +        if (!this.gotInfos) {
  +            if (this.isFile) {
                   File file = new File(systemId.substring(FILE.length()));
                   this.lastModificationDate = file.lastModified();
                   this.contentLength = file.length();
  @@ -91,7 +91,7 @@
                       if (this.connection == null) {
                           this.connection = this.url.openConnection();
                           String userInfo = this.getUserInfo();
  -                        if (this.url.getProtocol().startsWith("http") == true && 
userInfo != null) {
  +                        if (this.url.getProtocol().startsWith("http") && userInfo 
!= null) {
                               
this.connection.setRequestProperty("Authorization","Basic 
"+this.encodeBASE64(userInfo));
                           }
                       }
  @@ -136,14 +136,14 @@
           this.getInfos();
           try{
               InputStream input = null;
  -            if (this.isFile == true) {
  +            if (this.isFile) {
                   input = new FileInputStream(this.systemId.substring(FILE.length()));
               } else {
                   if (this.connection == null) {
                       this.connection = this.url.openConnection();
                       /* The following requires a jdk 1.3 */
                       String userInfo = this.getUserInfo();
  -                    if (this.url.getProtocol().startsWith("http") == true && 
userInfo != null) {
  +                    if (this.url.getProtocol().startsWith("http") && userInfo != 
null) {
                           this.connection.setRequestProperty("Authorization","Basic 
"+encodeBASE64(userInfo));
                       }
                   }
  @@ -168,8 +168,8 @@
        * method which is introduced in jdk 1.3
        */
       private String getUserInfo() {
  -        if (URLSource.checkedURLClass == true) {
  -            if (URLSource.urlSupportsGetUserInfo == true) {
  +        if (URLSource.checkedURLClass) {
  +            if (URLSource.urlSupportsGetUserInfo) {
                   try {
                       return (String) URLSource.urlGetUserInfo.invoke(this.url, 
URLSource.emptyParams);
                   } catch (Exception e){
  
  
  
  1.3       +10 -10    
xml-cocoon2/src/java/org/apache/cocoon/components/store/FilesystemQueueImpl.java
  
  Index: FilesystemQueueImpl.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/store/FilesystemQueueImpl.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- FilesystemQueueImpl.java  7 Jan 2002 12:39:14 -0000       1.2
  +++ FilesystemQueueImpl.java  22 Jan 2002 00:17:12 -0000      1.3
  @@ -71,7 +71,7 @@
           this.workdir = (File)context.get(Constants.CONTEXT_WORK_DIR);
           this.cachedirstr = IOUtils.getContextFilePath(this.workdir.getPath(),
                                                         this.cachedir.getPath());
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Context path="
                 + 
IOUtils.getContextFilePath(this.workdir.getPath(),this.cachedir.getPath()));
           }
  @@ -84,7 +84,7 @@
        */
       public void compose(ComponentManager manager) throws ComponentException {
           this.manager = manager;
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               getLogger().debug("Looking up " + Store.ROLE + "/Filesystem");
           }
           this.fsstore = (Store)manager.lookup(Store.ROLE + "/Filesystem");
  @@ -113,7 +113,7 @@
           this.handlerinterval = params.getParameterAsInteger("handlerinterval",10);
           this.threadpriority = params.getParameterAsInteger("threadpriority",5);
            this.maxobjects = params.getParameterAsInteger("maxobjects",100);
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Configure Filesystem Queue");
               this.getLogger().debug("handlerinterval=" + handlerinterval);
               this.getLogger().debug("threadpriority=" + threadpriority);
  @@ -155,7 +155,7 @@
        * Clear all elements from queue.
        */
       public void clear() {
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Clearing the FilesystemQueue");
           }
           ListIterator listIterator = linkedList.listIterator(0);
  @@ -180,7 +180,7 @@
        */
       public void insert(Object element) {
           if(this.linkedList.size() < maxobjects) {
  -            if (this.getLogger().isDebugEnabled() == true) {
  +            if (this.getLogger().isDebugEnabled()) {
                   this.getLogger().debug("Insert Element in FilesystemQueue");
               }
               this.linkedList.addFirst(element);
  @@ -194,10 +194,10 @@
        * Return element on top of heap but don't remove it.
        *
        * @return the element at top of heap
  -     * @exception NoSuchElementException if isEmpty() == true
  +     * @exception NoSuchElementException if queue is empty
        */
       public Object peek() throws NoSuchElementException {
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Peek Element in FilesystemQueue");
           }
           return (Comparable)linkedList.getLast();
  @@ -207,10 +207,10 @@
        * Return element on top of heap and remove it.
        *
        * @return the element at top of heap
  -     * @exception NoSuchElementException if isEmpty() == true
  +     * @exception NoSuchElementException if queue is empty
        */
       public Object pop() throws NoSuchElementException {
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Pop Element in FilesystemQueue");
           }
           return (Comparable)linkedList.removeLast();
  @@ -233,7 +233,7 @@
       }
   
       public void reportSize() {
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Size of FilesystemQueue=" + 
this.linkedList.size());
           }
       }
  
  
  
  1.2       +4 -4      
xml-cocoon2/src/java/org/apache/cocoon/components/store/FilesystemStore.java
  
  Index: FilesystemStore.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/store/FilesystemStore.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- FilesystemStore.java      3 Jan 2002 12:31:14 -0000       1.1
  +++ FilesystemStore.java      22 Jan 2002 00:17:12 -0000      1.2
  @@ -88,12 +88,12 @@
        * Get the file associated with the given unique key name.
        */
       public Object get(final Object key) {
  -        if (this.getLogger().isDebugEnabled() == true)
  +        if (this.getLogger().isDebugEnabled())
               getLogger().debug("FilesystemStore get(): Get file with key: " + 
key.toString());
           final File file = fileFromKey(key);
   
           if (file != null && file.exists()) {
  -            if (this.getLogger().isDebugEnabled() == true)
  +            if (this.getLogger().isDebugEnabled())
                   getLogger().debug("FilesystemStore get(): Found file with key: " + 
key.toString());
               return file;
           }
  @@ -180,7 +180,7 @@
           final int subStringBegin = this.directoryFile.getAbsolutePath().length() + 
1;
           final File[] files = directory.listFiles();
           for(int i=0; i<files.length; i++) {
  -            if (files[i].isDirectory() == true) {
  +            if (files[i].isDirectory()) {
                   this.addKeys(enum, files[i]);
               } else {
                   enum.add(files[i].getAbsolutePath().substring(subStringBegin));
  @@ -214,7 +214,7 @@
           }
   
           public Object nextElement() {
  -            if (this.hasMoreElements() == true) {
  +            if (this.hasMoreElements()) {
                   this.index++;
                   return this.array[index-1];
               }
  
  
  
  1.2       +5 -5      
xml-cocoon2/src/java/org/apache/cocoon/components/store/MRUMemoryStore.java
  
  Index: MRUMemoryStore.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/store/MRUMemoryStore.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- MRUMemoryStore.java       3 Jan 2002 12:31:14 -0000       1.1
  +++ MRUMemoryStore.java       22 Jan 2002 00:17:12 -0000      1.2
  @@ -93,7 +93,7 @@
           this.workdir = (File)context.get(Constants.CONTEXT_WORK_DIR);
           this.cachedirstr = IOUtils.getContextFilePath(this.workdir.getPath(),
                                                             this.cachedir.getPath());
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("cachedir=" + this.cachedir);
               this.getLogger().debug("workdir=" + this.workdir);
               this.getLogger().debug("cachedirstr=" + this.cachedirstr);
  @@ -338,7 +338,7 @@
               this.length = 0;
               this.index = 0;
   
  -            while (cache.hasMoreElements() == true) {
  +            while (cache.hasMoreElements()) {
                   this.add(cache.nextElement().toString());
               }
               RE re = null;
  @@ -349,9 +349,9 @@
                   ree.printStackTrace();
               }
   
  -            while (fs.hasMoreElements() == true) {
  +            while (fs.hasMoreElements()) {
                   final String key = fs.nextElement().toString();
  -                if (re.match(cachedir) == true) {
  +                if (re.match(cachedir)) {
                       this.add(key);
                   }
               }
  @@ -372,7 +372,7 @@
           }
   
           public Object nextElement() {
  -            if (this.hasMoreElements() == true) {
  +            if (this.hasMoreElements()) {
                   this.index++;
                   return this.array[index-1];
               }
  
  
  
  1.2       +14 -14    
xml-cocoon2/src/java/org/apache/cocoon/components/store/StoreJanitorImpl.java
  
  Index: StoreJanitorImpl.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/store/StoreJanitorImpl.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- StoreJanitorImpl.java     3 Jan 2002 12:31:14 -0000       1.1
  +++ StoreJanitorImpl.java     22 Jan 2002 00:17:12 -0000      1.2
  @@ -56,7 +56,7 @@
        * @exception ConfigurationException
        */
       public void configure(Configuration conf) throws ConfigurationException {
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Configure StoreJanitorImpl");
           }
           this.setJVM(Runtime.getRuntime());
  @@ -86,7 +86,7 @@
       public void start() {
           doRun = true;
           Thread checker = new Thread(this);
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Intializing checker thread");
           }
           checker.setPriority(this.getPriority());
  @@ -106,13 +106,13 @@
           while (doRun) {
               // amount of memory used is greater then heapsize
               if (this.memoryLow()) {
  -                if (this.getLogger().isDebugEnabled() == true) {
  +                if (this.getLogger().isDebugEnabled()) {
                       this.getLogger().debug("Invoking garbage collection, total 
memory = "
                               + this.getJVM().totalMemory() + ", free memory = "
                               + this.getJVM().freeMemory());
                   }
                   this.freePhysicalMemory();
  -                if (this.getLogger().isDebugEnabled() == true) {
  +                if (this.getLogger().isDebugEnabled()) {
                       this.getLogger().debug("Garbage collection complete, total 
memory = "
                           + this.getJVM().totalMemory() + ", free memory = "
                           + this.getJVM().freeMemory());
  @@ -136,7 +136,7 @@
        * @return true if memory is low
        */
       private boolean memoryLow() {
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("getJVM().totalMemory()=" + 
this.getJVM().totalMemory());
               this.getLogger().debug("getHeapsize()=" + this.getHeapsize());
               this.getLogger().debug("getJVM().freeMemory()=" + 
this.getJVM().freeMemory());
  @@ -152,7 +152,7 @@
        */
       public void register(Store store) {
           this.getStoreList().add(store);
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Registering store instance");
               this.getLogger().debug("Size of StoreJanitor now:" + 
this.getStoreList().size());
           }
  @@ -165,7 +165,7 @@
        */
       public void unregister(Store store) {
           this.getStoreList().remove(store);
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Unregister store instance");
               this.getLogger().debug("Size of StoreJanitor now:" + 
this.getStoreList().size());
           }
  @@ -192,27 +192,27 @@
        */
       private void freeMemory() {
           try {
  -            if (this.getLogger().isDebugEnabled() == true) {
  +            if (this.getLogger().isDebugEnabled()) {
                   this.getLogger().debug("StoreJanitor freeing memory!");
                   this.getLogger().debug("StoreList size=" + 
this.getStoreList().size());
                   this.getLogger().debug("Index before=" + this.getIndex());
               }
               if (this.getIndex() < this.getStoreList().size()) {
                   if(this.getIndex() == -1) {
  -                    if (this.getLogger().isDebugEnabled() == true) {
  +                    if (this.getLogger().isDebugEnabled()) {
                           this.getLogger().debug("Freeing at index=" + 
this.getIndex());
                       }
                       ((Store)this.getStoreList().get(0)).free();
                       this.setIndex(0);
                   } else {
  -                    if (this.getLogger().isDebugEnabled() == true) {
  +                    if (this.getLogger().isDebugEnabled()) {
                           this.getLogger().debug("Freeing at index=" + 
this.getIndex());
                       }
                       ((Store)this.getStoreList().get(this.getIndex())).free();
                       this.setIndex(this.getIndex() + 1);
                   }
               } else {
  -                if (this.getLogger().isDebugEnabled() == true) {
  +                if (this.getLogger().isDebugEnabled()) {
                       this.getLogger().debug("Starting from the beginning");
                   }
                   this.resetIndex();
  @@ -220,7 +220,7 @@
                   this.setIndex(0);
               }
               this.freePhysicalMemory();
  -            if (this.getLogger().isDebugEnabled() == true) {
  +            if (this.getLogger().isDebugEnabled()) {
                   this.getLogger().debug("Index after=" + this.getIndex());
               }
           } catch(Exception e) {
  @@ -285,7 +285,7 @@
       }
   
       private void setIndex(int _index) {
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Setting index=" + _index);
           }
           this.index = _index;
  @@ -296,7 +296,7 @@
       }
   
       private void resetIndex() {
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Reseting index");
           }
           this.index = -1;
  
  
  
  1.3       +3 -3      
xml-cocoon2/src/java/org/apache/cocoon/components/url/URLFactoryImpl.java
  
  Index: URLFactoryImpl.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/url/URLFactoryImpl.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- URLFactoryImpl.java       14 Jan 2002 12:11:52 -0000      1.2
  +++ URLFactoryImpl.java       22 Jan 2002 00:17:12 -0000      1.3
  @@ -34,7 +34,7 @@
   
   /**
    * @author <a href="mailto:[EMAIL PROTECTED]";>Giacomo Pati</a>
  - * @version $Id: URLFactoryImpl.java,v 1.2 2002/01/14 12:11:52 cziegeler Exp $
  + * @version $Id: URLFactoryImpl.java,v 1.3 2002/01/22 00:17:12 vgritsenko Exp $
    */
   public class URLFactoryImpl
   extends AbstractLoggable
  @@ -139,7 +139,7 @@
               String protocol = null;
               for (int i = 0; i < configs.length; i++) {
                   protocol = configs[i].getAttribute("name");
  -                if (factories.containsKey(protocol) == true) {
  +                if (factories.containsKey(protocol)) {
                       throw new ConfigurationException("URLFactory defined twice for 
protocol: " + protocol);
                   }
                   getLogger().debug("\tfor protocol: " + protocol + " " + 
configs[i].getAttribute("class"));
  @@ -169,7 +169,7 @@
       public void dispose() {
           Iterator iter = this.factories.values().iterator();
           URLFactory current;
  -        while (iter.hasNext() == true) {
  +        while (iter.hasNext()) {
               current = (URLFactory) iter.next();
               this.deinit(current);
           }
  
  
  
  1.2       +0 -1      
xml-cocoon2/src/java/org/apache/cocoon/components/xscript/XScriptManagerImpl.java
  
  Index: XScriptManagerImpl.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/xscript/XScriptManagerImpl.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- XScriptManagerImpl.java   3 Jan 2002 12:31:15 -0000       1.1
  +++ XScriptManagerImpl.java   22 Jan 2002 00:17:12 -0000      1.2
  @@ -26,7 +26,6 @@
   import org.apache.avalon.framework.configuration.Configuration;
   import org.apache.avalon.framework.configuration.ConfigurationException;
   
  -import org.apache.cocoon.components.xslt.XSLTProcessor;
   import org.apache.cocoon.components.xscript.XScriptObjectFromURL;
   
   /**
  
  
  
  1.4       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/components/xslt/XSLTProcessorImpl.java
  
  Index: XSLTProcessorImpl.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/components/xslt/XSLTProcessorImpl.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- XSLTProcessorImpl.java    14 Jan 2002 16:18:56 -0000      1.3
  +++ XSLTProcessorImpl.java    22 Jan 2002 00:17:12 -0000      1.4
  @@ -321,7 +321,7 @@
     {
       Templates templates = null;
   
  -    if (useStore == false)
  +    if (!useStore)
         return null;
   
       getLogger().debug("XSLTProcessorImpl getTemplates: stylesheet " + id);
  @@ -355,7 +355,7 @@
     private void putTemplates (Templates templates, Source stylesheet, String id)
       throws IOException, ProcessingException
     {
  -    if (useStore == false)
  +    if (!useStore)
         return;
   
       // only stylesheets with a last modification date are stored
  
  
  
  1.4       +3 -3      
xml-cocoon2/src/java/org/apache/cocoon/environment/AbstractEnvironment.java
  
  Index: AbstractEnvironment.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/environment/AbstractEnvironment.java,v
  retrieving revision 1.3
  retrieving revision 1.4
  diff -u -r1.3 -r1.4
  --- AbstractEnvironment.java  15 Jan 2002 07:42:02 -0000      1.3
  +++ AbstractEnvironment.java  22 Jan 2002 00:17:12 -0000      1.4
  @@ -26,7 +26,7 @@
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Giacomo Pati</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version CVS $Revision: 1.3 $ $Date: 2002/01/15 07:42:02 $
  + * @version CVS $Revision: 1.4 $ $Date: 2002/01/22 00:17:12 $
    */
   public abstract class AbstractEnvironment extends AbstractLoggable implements 
Environment {
   
  @@ -187,7 +187,7 @@
               getLogger().debug("\tfrom context(" + this.context.toExternalForm() + 
") and prefix(" + this.prefix + ")");
               getLogger().debug("\tat URI " + this.uris);
           }
  -        if (prefix.length() >= 1 && this.uris.startsWith(prefix) == false) {
  +        if (prefix.length() >= 1 && !this.uris.startsWith(prefix)) {
               if (getLogger().isErrorEnabled())
               {
                   this.getLogger().error("The current URI ("
  @@ -203,7 +203,7 @@
               this.uris = this.uris.substring(prefix.length());
   
               // check for a slash at the beginning to avoid problems with subsitemaps
  -            if (this.uris.startsWith("/") == true) {
  +            if (this.uris.startsWith("/")) {
                   this.uris = this.uris.substring(1);
                   this.prefix.append('/');
               }
  
  
  
  1.2       +6 -6      
xml-cocoon2/src/java/org/apache/cocoon/environment/commandline/AbstractCommandLineEnvironment.java
  
  Index: AbstractCommandLineEnvironment.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/environment/commandline/AbstractCommandLineEnvironment.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- AbstractCommandLineEnvironment.java       3 Jan 2002 12:31:16 -0000       1.1
  +++ AbstractCommandLineEnvironment.java       22 Jan 2002 00:17:12 -0000      1.2
  @@ -27,7 +27,7 @@
    * This environment is used to save the requested file to disk.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Stefano Mazzocchi</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:16 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:12 $
    */
   
   public abstract class AbstractCommandLineEnvironment
  @@ -54,11 +54,11 @@
        */
       public void redirect(boolean sessionmode, String newURL)
       throws IOException {
  -        if (sessionmode == true) {
  +        if (sessionmode) {
               CommandLineSession.getSession(true);
  -    }
  +        }
           // fix all urls created with request.getScheme()+... etc.
  -        if (newURL.startsWith("cli:/") == true) {
  +        if (newURL.startsWith("cli:/")) {
               int pos = newURL.indexOf('/', 6);
               newURL = newURL.substring(pos+1);
           }
  @@ -67,9 +67,9 @@
               newURL = "cocoon:/" + newURL;
           }
           // FIXME: this is a hack for the links view
  -        if (newURL.startsWith("cocoon:") == true
  +        if (newURL.startsWith("cocoon:")
               && this.getView() != null
  -            && this.getView().equals(Constants.LINK_VIEW) == true) {
  +            && this.getView().equals(Constants.LINK_VIEW)) {
   
               // as the internal cocoon protocol is used the last
               // serializer is removed from it! And therefore
  
  
  
  1.2       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/environment/commandline/CommandLineSession.java
  
  Index: CommandLineSession.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/environment/commandline/CommandLineSession.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- CommandLineSession.java   3 Jan 2002 12:31:16 -0000       1.1
  +++ CommandLineSession.java   22 Jan 2002 00:17:12 -0000      1.2
  @@ -18,7 +18,7 @@
    * Command-line version of Http Session.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Vadim Gritsenko</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:16 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:12 $
    */
   public final class CommandLineSession
   implements Session {
  @@ -81,7 +81,7 @@
        * Get the current session object - if available
        */
       public static Session getSession(boolean create) {
  -        if (create == true && session == null) {
  +        if (create && session == null) {
               session = new CommandLineSession();
           }
           return session;
  
  
  
  1.3       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/environment/commandline/LinkSamplingEnvironment.java
  
  Index: LinkSamplingEnvironment.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/environment/commandline/LinkSamplingEnvironment.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- LinkSamplingEnvironment.java      9 Jan 2002 21:30:10 -0000       1.2
  +++ LinkSamplingEnvironment.java      22 Jan 2002 00:17:12 -0000      1.3
  @@ -20,7 +20,7 @@
    * This environment is sample the links of the resource.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Stefano Mazzocchi</a>
  - * @version CVS $Revision: 1.2 $ $Date: 2002/01/09 21:30:10 $
  + * @version CVS $Revision: 1.3 $ $Date: 2002/01/22 00:17:12 $
    */
   
   public class LinkSamplingEnvironment extends AbstractCommandLineEnvironment
  @@ -48,7 +48,7 @@
        */
       public void setContentType( String contentType )
       {
  -        if ( Constants.LINK_CONTENT_TYPE.equals( contentType ) == false )
  +        if ( ! Constants.LINK_CONTENT_TYPE.equals( contentType ) )
           {
               this.skip = true;
           }
  
  
  
  1.2       +1 -1      
xml-cocoon2/src/java/org/apache/cocoon/environment/http/HttpContext.java
  
  Index: HttpContext.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/environment/http/HttpContext.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- HttpContext.java  3 Jan 2002 12:31:16 -0000       1.1
  +++ HttpContext.java  22 Jan 2002 00:17:13 -0000      1.2
  @@ -53,7 +53,7 @@
       }
   
       public String getRealPath(String path) {
  -        if (path.equals("/") == true) {
  +        if (path.equals("/")) {
               String value = servletContext.getRealPath(path);
               if (value == null) {
                   // Try to figure out the path of the root from that of WEB-INF
  
  
  
  1.3       +3 -3      
xml-cocoon2/src/java/org/apache/cocoon/environment/wrapper/EnvironmentWrapper.java
  
  Index: EnvironmentWrapper.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/environment/wrapper/EnvironmentWrapper.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- EnvironmentWrapper.java   21 Jan 2002 11:00:34 -0000      1.2
  +++ EnvironmentWrapper.java   22 Jan 2002 00:17:13 -0000      1.3
  @@ -30,7 +30,7 @@
    * contains a <code>RequestWrapper</code> object.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version $Id: EnvironmentWrapper.java,v 1.2 2002/01/21 11:00:34 cziegeler Exp $
  + * @version $Id: EnvironmentWrapper.java,v 1.3 2002/01/22 00:17:13 vgritsenko Exp $
    */
   public final class EnvironmentWrapper
   extends AbstractEnvironment
  @@ -77,7 +77,7 @@
           Map oldObjectModel = env.getObjectModel();
           Iterator keys = oldObjectModel.keySet().iterator();
           String key;
  -        while (keys.hasNext() == true) {
  +        while (keys.hasNext()) {
               key = (String)keys.next();
               this.objectModel.put(key, oldObjectModel.get(key));
           }
  @@ -95,7 +95,7 @@
           this.redirectURL = newURL;
   
           // check if session mode shall be activated
  -        if (sessionmode == true) {
  +        if (sessionmode) {
               // The session
               Session session = null;
               // get session from request, or create new session
  
  
  
  1.2       +6 -6      
xml-cocoon2/src/java/org/apache/cocoon/environment/wrapper/RequestParameters.java
  
  Index: RequestParameters.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/environment/wrapper/RequestParameters.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- RequestParameters.java    3 Jan 2002 12:31:16 -0000       1.1
  +++ RequestParameters.java    22 Jan 2002 00:17:13 -0000      1.2
  @@ -16,7 +16,7 @@
    * for the <code>Request</code> object.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version $Id: RequestParameters.java,v 1.1 2002/01/03 12:31:16 giacomo Exp $
  + * @version $Id: RequestParameters.java,v 1.2 2002/01/22 00:17:13 vgritsenko Exp $
    */
   public final class RequestParameters
   implements Serializable {
  @@ -84,7 +84,7 @@
        */
       private void setParameter(String name, String value) {
           ArrayList list;
  -        if (names.containsKey(name) == true) {
  +        if (names.containsKey(name)) {
               list = (ArrayList)names.get(name);
           } else {
               list = new ArrayList(3);
  @@ -100,7 +100,7 @@
        *               or <CODE>null</CODE>
        */
       public String getParameter(String name) {
  -        if (names.containsKey(name) == true) {
  +        if (names.containsKey(name)) {
               return (String)((ArrayList)names.get(name)).get(0);
           }
           return null;
  @@ -114,7 +114,7 @@
        *               or <CODE>defaultValue</CODE>
        */
       public String getParameter(String name, String defaultValue) {
  -        if (names.containsKey(name) == true) {
  +        if (names.containsKey(name)) {
               return (String)((ArrayList)names.get(name)).get(0);
           }
           return defaultValue;
  @@ -127,11 +127,11 @@
        *               is not defined.
        */
       public String[] getParameterValues(String name) {
  -        if (names.containsKey(name) == true) {
  +        if (names.containsKey(name)) {
               String values[] = null;
               ArrayList list = (ArrayList)names.get(name);
               Iterator iter = list.iterator();
  -            while (iter.hasNext() == true) {
  +            while (iter.hasNext()) {
                   if (values == null) {
                       values = new String[1];
                   } else {
  
  
  
  1.3       +3 -3      
xml-cocoon2/src/java/org/apache/cocoon/environment/wrapper/RequestWrapper.java
  
  Index: RequestWrapper.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/environment/wrapper/RequestWrapper.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- RequestWrapper.java       21 Jan 2002 11:00:34 -0000      1.2
  +++ RequestWrapper.java       22 Jan 2002 00:17:13 -0000      1.3
  @@ -23,7 +23,7 @@
    * are different.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version $Id: RequestWrapper.java,v 1.2 2002/01/21 11:00:34 cziegeler Exp $
  + * @version $Id: RequestWrapper.java,v 1.3 2002/01/22 00:17:13 vgritsenko Exp $
    */
   public final class RequestWrapper implements Request {
   
  @@ -94,11 +94,11 @@
           // put all parameter names into a set
           Set parameterNames = new HashSet();
           Enumeration names = this.parameters.getParameterNames();
  -        while (names.hasMoreElements() == true) {
  +        while (names.hasMoreElements()) {
               parameterNames.add(names.nextElement());
           }
           names = this.req.getParameterNames();
  -        while (names.hasMoreElements() == true) {
  +        while (names.hasMoreElements()) {
               parameterNames.add(names.nextElement());
           }
           return new EnumerationFromIterator(parameterNames.iterator());
  
  
  
  1.3       +9 -17     
xml-cocoon2/src/java/org/apache/cocoon/i18n/XMLResourceBundle.java
  
  Index: XMLResourceBundle.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/i18n/XMLResourceBundle.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- XMLResourceBundle.java    8 Jan 2002 05:44:54 -0000       1.2
  +++ XMLResourceBundle.java    22 Jan 2002 00:17:13 -0000      1.3
  @@ -32,7 +32,7 @@
    * @author <a href="mailto:[EMAIL PROTECTED]";>Mike Engelhart</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Neeme Praks</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Oleg Podolsky</a>
  - * @version $Id: XMLResourceBundle.java,v 1.2 2002/01/08 05:44:54 vgritsenko Exp $
  + * @version $Id: XMLResourceBundle.java,v 1.3 2002/01/22 00:17:13 vgritsenko Exp $
    */
   public class XMLResourceBundle
       extends ResourceBundle
  @@ -318,16 +318,12 @@
        */
       private String _getString(Node node, String key)
       {
  -        String value = null;
  -        try
  -        {
  -            value = getTextValue(_getNode(node, key));
  -        }
  -        catch (Exception e)
  -        {
  +        try {
  +            return getTextValue(_getNode(node, key));
  +        } catch (Exception e) {
               logger.error(name + ": error while locating resource: " + key, e);
           }
  -        return value;
  +        return null;
       }
   
       /**
  @@ -385,16 +381,12 @@
        */
       private Node _getNode(Node rootNode, String key)
       {
  -        Node node = null;
  -        try
  -        {
  -            node = this.processor.selectSingleNode(rootNode, key);
  -        }
  -        catch (Exception e)
  -        {
  +        try {
  +            return this.processor.selectSingleNode(rootNode, key);
  +        } catch (Exception e) {
               logger.error("Error while locating resource with key: " + key, e);
           }
  -        return node;
  +        return null;
       }
   
       /**
  
  
  
  1.3       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/reading/DatabaseReader.java
  
  Index: DatabaseReader.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/reading/DatabaseReader.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- DatabaseReader.java       8 Jan 2002 10:47:45 -0000       1.2
  +++ DatabaseReader.java       22 Jan 2002 00:17:13 -0000      1.3
  @@ -84,14 +84,14 @@
               this.datasource = (DataSourceComponent) dbselector.select(dsn);
               this.con = datasource.getConnection();
   
  -            if (this.con.getAutoCommit() == true) {
  +            if (this.con.getAutoCommit()) {
                   this.con.setAutoCommit(false);
               }
   
               PreparedStatement statement = con.prepareStatement(getQuery());
               statement.setString(1, this.source);
               ResultSet set = statement.executeQuery();
  -            if (set.next() == false) throw new ResourceNotFoundException("There is 
no resource with that key");
  +            if (!set.next()) throw new ResourceNotFoundException("There is no 
resource with that key");
   
               Response response = (Response) 
objectModel.get(Constants.RESPONSE_OBJECT);
               Request request = (Request) objectModel.get(Constants.REQUEST_OBJECT);
  
  
  
  1.3       +4 -4      
xml-cocoon2/src/java/org/apache/cocoon/servlet/CocoonServlet.java
  
  Index: CocoonServlet.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/servlet/CocoonServlet.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- CocoonServlet.java        9 Jan 2002 08:16:28 -0000       1.2
  +++ CocoonServlet.java        22 Jan 2002 00:17:13 -0000      1.3
  @@ -66,7 +66,7 @@
    * @author <a href="mailto:[EMAIL PROTECTED]";>Berin Loritsch</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Leo Sutic</a>
  - * @version CVS $Revision: 1.2 $ $Date: 2002/01/09 08:16:28 $
  + * @version CVS $Revision: 1.3 $ $Date: 2002/01/22 00:17:13 $
    */
   public class CocoonServlet extends HttpServlet {
   
  @@ -451,7 +451,7 @@
        protected String getExtraClassPath()
        throws ServletException {
            String extraClassPath = this.getInitParameter("extra-classpath");
  -         if ((extraClassPath != null) && (extraClassPath.trim().equals("") == 
false)) {
  +         if ((extraClassPath != null) && !extraClassPath.trim().equals("")) {
                StringBuffer sb = new StringBuffer();
                StringTokenizer st = new StringTokenizer(extraClassPath, 
System.getProperty("path.separator"), false);
                int i = 0;
  @@ -1035,7 +1035,7 @@
        */
       private Cocoon getCocoon(final String pathInfo, final String reloadParam)
       throws ServletException {
  -        if (this.cocoon != null && this.allowReload == true) {
  +        if (this.cocoon != null && this.allowReload) {
               if (this.cocoon.modifiedSince(this.creationTime)) {
                   if (log.isInfoEnabled()) {
                       log.info("Configuration changed reload attempt");
  @@ -1051,7 +1051,7 @@
                   this.createCocoon();
                   return this.cocoon;
               }
  -        } else if ((pathInfo == null) && (this.allowReload == true) && (reloadParam 
!= null)) {
  +        } else if ((pathInfo == null) && this.allowReload && (reloadParam != null)) 
{
               if (log.isInfoEnabled()) {
                   log.info("Invalid configurations reload");
               }
  
  
  
  1.2       +3 -3      
xml-cocoon2/src/java/org/apache/cocoon/servlet/ParanoidCocoonServlet.java
  
  Index: ParanoidCocoonServlet.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/servlet/ParanoidCocoonServlet.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- ParanoidCocoonServlet.java        3 Jan 2002 12:31:20 -0000       1.1
  +++ ParanoidCocoonServlet.java        22 Jan 2002 00:17:13 -0000      1.2
  @@ -22,7 +22,7 @@
    * of it.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Berin Loritsch</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:20 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:13 $
    */
   
   public class ParanoidCocoonServlet extends CocoonServlet {
  @@ -52,13 +52,13 @@
           String classDir;
           File root;
   
  -        if ((classDirPath != null) && (classDirPath.trim().equals("") == false)) {
  +        if ((classDirPath != null) && !classDirPath.trim().equals("")) {
               classDir = classDirPath;
           } else {
               classDir = this.servletContext.getRealPath("/WEB-INF/classes");
           }
   
  -        if ((libDirPath != null) && (libDirPath.trim().equals("") == false)) {
  +        if ((libDirPath != null) && !libDirPath.trim().equals("")) {
               root = new File(libDirPath);
           } else {
               root = new File(this.servletContext.getRealPath("/WEB-INF/lib"));
  
  
  
  1.2       +4 -4      
xml-cocoon2/src/java/org/apache/cocoon/sitemap/ContentAggregator.java
  
  Index: ContentAggregator.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/sitemap/ContentAggregator.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- ContentAggregator.java    3 Jan 2002 12:31:20 -0000       1.1
  +++ ContentAggregator.java    22 Jan 2002 00:17:13 -0000      1.2
  @@ -33,7 +33,7 @@
   /**
    * @author <a href="mailto:[EMAIL PROTECTED]";>Giacomo Pati</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version CVS $Id: ContentAggregator.java,v 1.1 2002/01/03 12:31:20 giacomo Exp $
  + * @version CVS $Id: ContentAggregator.java,v 1.2 2002/01/22 00:17:13 vgritsenko 
Exp $
    */
   public class ContentAggregator
   extends ContentHandlerWrapper
  @@ -86,7 +86,7 @@
        */
       public void generate()
       throws IOException, SAXException, ProcessingException {
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               getLogger().debug("ContentAggregator: generating aggregated content");
           }
           this.contentHandler.startDocument();
  @@ -191,7 +191,7 @@
           this.rootElement = new Element(element,
                                          namespace,
                                          prefix);
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("ContentAggregator: root element='" + element +
                                      "' ns='" + namespace + "' prefix='" + prefix + 
"'");
           }
  @@ -220,7 +220,7 @@
           this.parts.add(new Part(uri,
                                   elem,
                                   stripRootElement));
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("ContentAggregator: part uri='" + uri +
                                      "' element='" + element + "' ns='" + namespace +
                                      "' stripRootElement='" + stripRootElement + "' 
prefix='" + prefix + "'");
  
  
  
  1.3       +2 -2      
xml-cocoon2/src/java/org/apache/cocoon/sitemap/SitemapComponentSelector.java
  
  Index: SitemapComponentSelector.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/sitemap/SitemapComponentSelector.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- SitemapComponentSelector.java     15 Jan 2002 11:10:52 -0000      1.2
  +++ SitemapComponentSelector.java     22 Jan 2002 00:17:13 -0000      1.3
  @@ -23,7 +23,7 @@
   /** Default component manager for Cocoon's sitemap components.
    * @author <a href="mailto:[EMAIL PROTECTED]";>Berin Loritsch</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Giacomo Pati</a>
  - * @version CVS $Id: SitemapComponentSelector.java,v 1.2 2002/01/15 11:10:52 
sylvain Exp $
  + * @version CVS $Id: SitemapComponentSelector.java,v 1.3 2002/01/22 00:17:13 
vgritsenko Exp $
    */
   public class SitemapComponentSelector extends ExcaliburComponentSelector
     implements OutputComponentSelector {
  @@ -80,7 +80,7 @@
       }
           
       public String getMimeTypeForHint(Object hint) {
  -        if (this.mime_types.containsKey(hint) == true) {
  +        if (this.mime_types.containsKey(hint)) {
               return (String)this.mime_types.get(hint);
           }
           if (this.parentSelector != null) {
  
  
  
  1.3       +3 -3      
xml-cocoon2/src/java/org/apache/cocoon/sitemap/SitemapManager.java
  
  Index: SitemapManager.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/sitemap/SitemapManager.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- SitemapManager.java       19 Jan 2002 00:51:45 -0000      1.2
  +++ SitemapManager.java       22 Jan 2002 00:17:13 -0000      1.3
  @@ -38,7 +38,7 @@
    * to Java code.
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Sylvain Wallez</a>
  - * @version CVS $Revision: 1.2 $ $Date: 2002/01/19 00:51:45 $
  + * @version CVS $Revision: 1.3 $ $Date: 2002/01/22 00:17:13 $
    */
   public class SitemapManager extends Manager implements Processor, Configurable {
       
  @@ -58,10 +58,10 @@
           this.sitemapFileName = sconf.getAttribute("file");
   
           String value = sconf.getAttribute("check-reload", "yes");
  -        this.checkSitemapReload = !(value != null && value.equalsIgnoreCase("no") 
== true);
  +        this.checkSitemapReload = !(value != null && value.equalsIgnoreCase("no"));
           
           value = sconf.getAttribute("reload-method", "asynchron");
  -        this.reloadSitemapAsynchron = !(value != null && 
value.equalsIgnoreCase("synchron") == true);
  +        this.reloadSitemapAsynchron = !(value != null && 
value.equalsIgnoreCase("synchron"));
   
           long checkDelay = sconf.getAttributeAsLong("check-delay", 1L);
           Handler.setSitemapCheckDelay(checkDelay * 1000L);
  
  
  
  1.6       +4 -5      
xml-cocoon2/src/java/org/apache/cocoon/transformation/TraxTransformer.java
  
  Index: TraxTransformer.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/transformation/TraxTransformer.java,v
  retrieving revision 1.5
  retrieving revision 1.6
  diff -u -r1.5 -r1.6
  --- TraxTransformer.java      17 Jan 2002 20:14:32 -0000      1.5
  +++ TraxTransformer.java      22 Jan 2002 00:17:13 -0000      1.6
  @@ -113,7 +113,7 @@
    * @author <a href="mailto:[EMAIL PROTECTED]";>Giacomo Pati</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Ovidiu Predescu</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Mark H. Butler</a>
  - * @version CVS $Id: TraxTransformer.java,v 1.5 2002/01/17 20:14:32 vgritsenko Exp $
  + * @version CVS $Id: TraxTransformer.java,v 1.6 2002/01/22 00:17:13 vgritsenko Exp $
    */
   public class TraxTransformer extends AbstractTransformer
   implements Transformer, Composable, Recyclable, Configurable, Cacheable, Disposable 
{
  @@ -191,7 +191,7 @@
             this.useDeli = child.getValueAsBoolean(false);
             this._useDeli = this.useDeli;
   
  -          if (this.getLogger().isDebugEnabled() == true) {
  +          if (this.getLogger().isDebugEnabled()) {
                   this.getLogger().debug("Use parameters is " + this.useParameters + 
" for " + this);
                   this.getLogger().debug("Use cookies is " + this.useCookies + " for 
" + this);
                   this.getLogger().debug("Use browser capabilities is " + 
this.useBrowserCap + " for " + this);
  @@ -251,7 +251,7 @@
           _useDeli = par.getParameterAsBoolean("use-deli", this.useDeli);
           _useCookies = par.getParameterAsBoolean("use-cookies", this.useCookies);
           _useSessionInfo = par.getParameterAsBoolean("use-session-info", 
this.useSessionInfo);
  -        if (this.getLogger().isDebugEnabled() == true) {
  +        if (this.getLogger().isDebugEnabled()) {
               this.getLogger().debug("Using stylesheet: 
'"+this.inputSource.getSystemId()+"' in " + this + ", last modified: " + 
this.inputSource.getLastModified());
           }
   
  @@ -461,8 +461,7 @@
                       /* add the accept param */
                       map.put("accept", accept);
                   } else {
  -                    // FIXME (VG): Is this an error condition?
  -                    getLogger().warn("Deli is not available! Capabilities are not 
set.");
  +                    getLogger().warn("TraxTransformer configured to use DELI in 
sitemap.xmap but DELI not enabled in cocoon.xconf.");
                   }
   
               } catch (Exception e) {
  
  
  
  1.2       +4 -4      
xml-cocoon2/src/java/org/apache/cocoon/util/log/XMLCocoonLogFormatter.java
  
  Index: XMLCocoonLogFormatter.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/util/log/XMLCocoonLogFormatter.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- XMLCocoonLogFormatter.java        3 Jan 2002 12:31:22 -0000       1.1
  +++ XMLCocoonLogFormatter.java        22 Jan 2002 00:17:13 -0000      1.2
  @@ -50,7 +50,7 @@
    * @author <a href="mailto:[EMAIL PROTECTED]";>Peter Donald</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Sylvain Wallez</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Carsten Ziegeler</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:22 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/22 00:17:13 $
    */
   public class XMLCocoonLogFormatter
   implements Formatter {
  @@ -265,14 +265,14 @@
       protected int getTypeIdFor( final String type ) {
           int index = 0;
           boolean found = false;
  -        while (found == false && index < typeStrings.length) {
  -            if (type.equalsIgnoreCase(typeStrings[index]) == true) {
  +        while (!found && index < typeStrings.length) {
  +            if (type.equalsIgnoreCase(typeStrings[index])) {
                   found = true;
               } else {
                   index++;
               }
           }
  -        if (found == true) return index;
  +        if (found) return index;
           throw new IllegalArgumentException( "Unknown Type - " + type );
       }
   
  
  
  

----------------------------------------------------------------------
In case of troubles, e-mail:     [EMAIL PROTECTED]
To unsubscribe, e-mail:          [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]

Reply via email to