You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@cocoon.apache.org by an...@apache.org on 2006/05/29 10:52:44 UTC

svn commit: r410057 - /cocoon/trunk/core/cocoon-core/src/main/java/org/apache/cocoon/acting/CopySourceAction.java

Author: antonio
Date: Mon May 29 01:52:43 2006
New Revision: 410057

URL: http://svn.apache.org/viewvc?rev=410057&view=rev
Log:
Using SourceUtil.copy(). Thanks to Frédéric Glorieux. See COCOON-1815

Modified:
    cocoon/trunk/core/cocoon-core/src/main/java/org/apache/cocoon/acting/CopySourceAction.java

Modified: cocoon/trunk/core/cocoon-core/src/main/java/org/apache/cocoon/acting/CopySourceAction.java
URL: http://svn.apache.org/viewvc/cocoon/trunk/core/cocoon-core/src/main/java/org/apache/cocoon/acting/CopySourceAction.java?rev=410057&r1=410056&r2=410057&view=diff
==============================================================================
--- cocoon/trunk/core/cocoon-core/src/main/java/org/apache/cocoon/acting/CopySourceAction.java (original)
+++ cocoon/trunk/core/cocoon-core/src/main/java/org/apache/cocoon/acting/CopySourceAction.java Mon May 29 01:52:43 2006
@@ -28,6 +28,7 @@
 import org.apache.excalibur.source.ModifiableSource;
 import org.apache.excalibur.source.Source;
 import org.apache.excalibur.source.SourceResolver;
+import org.apache.excalibur.source.SourceUtil;
 import org.apache.excalibur.source.TraversableSource;
 
 /**
@@ -89,17 +90,9 @@
 
         // And transfer all content.
         try {
-            byte[] buffer = new byte[1024];
-            int len;
-            while ((len = is.read(buffer, 0, buffer.length)) > 0) {
-                os.write(buffer, 0, len);
-            }
-            os.close();
-        } catch(Exception e) {
-            if (wdest.canCancel(os)) {
-                wdest.cancel(os);
-            }
+            SourceUtil.copy(is, os);
         } finally {
+            os.close();
             is.close();
         }
         // Success !