vgritsenko    02/01/24 19:48:30

  Modified:    src/java/org/apache/cocoon/generation ScriptGenerator.java
                        ServerPagesGenerator.java VelocityGenerator.java
  Log:
  code formatting
  
  Revision  Changes    Path
  1.3       +1 -4      
xml-cocoon2/src/java/org/apache/cocoon/generation/ScriptGenerator.java
  
  Index: ScriptGenerator.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/generation/ScriptGenerator.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- ScriptGenerator.java      3 Jan 2002 14:56:56 -0000       1.2
  +++ ScriptGenerator.java      25 Jan 2002 03:48:30 -0000      1.3
  @@ -45,7 +45,7 @@
    * </pre>
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Jason Foster</a>
  - * @version CVS $Revision: 1.2 $ $Date: 2002/01/03 14:56:56 $
  + * @version CVS $Revision: 1.3 $ $Date: 2002/01/25 03:48:30 $
    */
   public class ScriptGenerator extends ComposerGenerator implements Configurable,
   Recyclable {
  @@ -111,7 +111,6 @@
           Parser parser = null;
           try {
               // Figure out what file to open and do so
  -
               getLogger().debug("processing file [" + super.source + "]");
               this.inputSource = this.resolver.resolve(super.source);
   
  @@ -119,11 +118,9 @@
   
               // TODO: why doesn't this work?
               // Reader in = src.getCharacterStream();
  -
               Reader in = new 
java.io.InputStreamReader(this.inputSource.getInputStream());
   
               // Set up the BSF manager and register relevant helper "beans"
  -
               BSFManager mgr = new BSFManager();
   
               // add support for additional languages
  
  
  
  1.3       +6 -4      
xml-cocoon2/src/java/org/apache/cocoon/generation/ServerPagesGenerator.java
  
  Index: ServerPagesGenerator.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/generation/ServerPagesGenerator.java,v
  retrieving revision 1.2
  retrieving revision 1.3
  diff -u -r1.2 -r1.3
  --- ServerPagesGenerator.java 4 Jan 2002 13:24:09 -0000       1.2
  +++ ServerPagesGenerator.java 25 Jan 2002 03:48:30 -0000      1.3
  @@ -59,7 +59,7 @@
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Ricardo Rocha</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Sylvain Wallez</a>
  - * @version CVS $Revision: 1.2 $ $Date: 2002/01/04 13:24:09 $
  + * @version CVS $Revision: 1.3 $ $Date: 2002/01/25 03:48:30 $
    */
   public class ServerPagesGenerator extends ServletGenerator
     implements Recyclable, Disposable, Cacheable, Configurable
  @@ -255,15 +255,17 @@
        */
       public void recycle() {
           super.recycle();
  -        if (generator != null)
  -            programGenerator.release(generator);
  -        this.generator = null;
  +        if (this.generator != null) {
  +            programGenerator.release(this.generator);
  +            this.generator = null;
  +        }
           if (this.inputSource != null) {
               this.inputSource.recycle();
               this.inputSource = null;
           }
           if (this.completionPipe != null) {
               this.completionPipe.recycle();
  +            this.completionPipe = null;
           }
       }
   
  
  
  
  1.2       +28 -58    
xml-cocoon2/src/java/org/apache/cocoon/generation/VelocityGenerator.java
  
  Index: VelocityGenerator.java
  ===================================================================
  RCS file: 
/home/cvs/xml-cocoon2/src/java/org/apache/cocoon/generation/VelocityGenerator.java,v
  retrieving revision 1.1
  retrieving revision 1.2
  diff -u -r1.1 -r1.2
  --- VelocityGenerator.java    3 Jan 2002 12:31:16 -0000       1.1
  +++ VelocityGenerator.java    25 Jan 2002 03:48:30 -0000      1.2
  @@ -125,7 +125,7 @@
    *
    * @author <a href="mailto:[EMAIL PROTECTED]";>Davanum Srinivas</a>
    * @author <a href="mailto:[EMAIL PROTECTED]";>Michael McKibben</a>
  - * @version CVS $Revision: 1.1 $ $Date: 2002/01/03 12:31:16 $
  + * @version CVS $Revision: 1.2 $ $Date: 2002/01/25 03:48:30 $
    */
   public class VelocityGenerator extends ComposerGenerator 
       implements Initializable, Configurable, Recyclable, LogSystem 
  @@ -228,31 +228,22 @@
            * @param systemId the path to the resource
            */
           private Source resolveSource(String systemId) throws 
org.apache.velocity.exception.ResourceNotFoundException 
  -     {
  +        {
               Map sourceCache;
  -            try 
  -         {
  +            try {
                   sourceCache = 
(Map)this.resolverContext.get(CONTEXT_SOURCE_CACHE_KEY);
  -            } 
  -         catch (ContextException ignore) 
  -         {
  +            } catch (ContextException ignore) {
                   throw new 
org.apache.velocity.exception.ResourceNotFoundException("Runtime Cocoon source cache 
not specified in resource loader resolver context.");
               }
   
               Source source = (Source)sourceCache.get(systemId);
  -            if (source == null) 
  -         {
  -                try 
  -             {
  +            if (source == null) {
  +                try {
                       SourceResolver resolver = 
(SourceResolver)this.resolverContext.get(CONTEXT_RESOLVER_KEY);
                       source = resolver.resolve(systemId);
  -                } 
  -             catch (ContextException ex) 
  -             {
  +                } catch (ContextException ex) {
                       throw new 
org.apache.velocity.exception.ResourceNotFoundException("No Cocoon source resolver 
associated with current request.");
  -                } 
  -             catch (Exception ex) 
  -             {
  +                } catch (Exception ex) {
                       throw new 
org.apache.velocity.exception.ResourceNotFoundException("Unable to resolve source: " + 
ex);
                   }
               }
  @@ -450,20 +441,16 @@
        */
       public void recycle() 
       {
  -     this.activeFlag = false;
  +        this.activeFlag = false;
   
           // recycle all the Source objects resolved/used by our resource loader
           //
  -        try 
  -     {
  +        try {
               Map sourceCache = 
(Map)this.resolverContext.get(CONTEXT_SOURCE_CACHE_KEY);
  -            for (Iterator it = sourceCache.values().iterator(); it.hasNext();) 
  -         {
  +            for (Iterator it = sourceCache.values().iterator(); it.hasNext();) {
                   ((Source)it.next()).recycle();
               }
  -        } 
  -     catch (ContextException ignore) 
  -     {
  +        } catch (ContextException ignore) {
           }
   
           this.velocityContext = null;
  @@ -478,19 +465,15 @@
       public void generate()
       throws IOException, SAXException, ProcessingException 
       {
  -     // Guard against calling generate before setup.
  -     //
  -     if (!activeFlag) 
  -     {
  -         throw new IllegalStateException("generate called on sitemap component 
before setup.");
  -     }
  +        // Guard against calling generate before setup.
  +        if (!activeFlag) {
  +            throw new IllegalStateException("generate called on sitemap component 
before setup.");
  +        }
   
           Parser parser = null;
  -        try 
  -     {
  -            parser = (Parser)(this.manager.lookup(Parser.ROLE));
  -            if (getLogger().isDebugEnabled()) 
  -         {
  +        try {
  +            parser = (Parser)this.manager.lookup(Parser.ROLE);
  +            if (getLogger().isDebugEnabled()) {
                   getLogger().debug("Processing File: " + super.source);
               }
   
  @@ -502,37 +485,24 @@
               new InputSource(new StringReader(w.toString()));
               parser.setConsumer(this.xmlConsumer);
               parser.parse(xmlInput);
  -        } 
  -     catch (IOException e) 
  -     {
  +        } catch (IOException e) {
               getLogger().warn("VelocityGenerator.generate()", e);
               throw new ResourceNotFoundException("Could not get Resource for 
VelocityGenerator", e);
  -        } 
  -     catch (SAXException e) 
  -     {
  +        } catch (SAXException e) {
               getLogger().error("VelocityGenerator.generate()", e);
               throw e;
  -        } 
  -     catch (ComponentException e) 
  -     {
  +        } catch (ComponentException e) {
               getLogger().error("Could not get parser", e);
               throw new ProcessingException("Exception in 
VelocityGenerator.generate()", e);
  -        } 
  -     catch (ProcessingException e) 
  -     {
  +        } catch (ProcessingException e) {
               throw e;
  -        } 
  -     catch (Exception e) 
  -     {
  +        } catch (Exception e) {
               getLogger().error("Could not get parser", e);
               throw new ProcessingException("Exception in 
VelocityGenerator.generate()", e);
  -        } 
  -     finally 
  -     {
  -            if (parser != null) 
  -         {
  -             this.manager.release(parser);
  -         }
  +        } finally {
  +            if (parser != null) {
  +                this.manager.release(parser);
  +            }
           }
       }
   
  
  
  

----------------------------------------------------------------------
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