Re: [shale] Patch for org.apache.shale.remote.ResponseWrapper

[Shane Bryzak]

Thanks for explaining the process Craig,

I've added the patch to bugzilla, #35106.

Shane

Craig McClanahan wrote:


Shane,

(Late response due to just returning from vacation).

It's certainly on topic to post about Shale here -- but the best way
to ensure that patches don't get lost is to file them in our issue
tracking system:

 http://issues.apache.org/bugzilla/

The patch can either be listed directly in the bug report (if it's
simple like this one), or added as an attachment if it's larger.

Craig


On 5/23/05, Shane Bryzak [EMAIL PROTECTED] wrote:
 


I hope this is the right place to post patches... anyway, I found a
small bug in the ResponseWrapper class.  Patch as follows.

Index: ResponseWrapper.java
===
--- ResponseWrapper.java(revision 177960)
+++ ResponseWrapper.java(working copy)
@@ -259,7 +259,7 @@

if (open) {
writer.write(/);
-writer.close();
+close();
} else {
writer.write(/);
writer.write(name);


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


   



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




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

Re: [shale] Patch for org.apache.shale.remote.ResponseWrapper

[Craig McClanahan]

Shane,

(Late response due to just returning from vacation).

It's certainly on topic to post about Shale here -- but the best way
to ensure that patches don't get lost is to file them in our issue
tracking system:

  http://issues.apache.org/bugzilla/

The patch can either be listed directly in the bug report (if it's
simple like this one), or added as an attachment if it's larger.

Craig


On 5/23/05, Shane Bryzak [EMAIL PROTECTED] wrote:
 I hope this is the right place to post patches... anyway, I found a
 small bug in the ResponseWrapper class.  Patch as follows.
 
 Index: ResponseWrapper.java
 ===
 --- ResponseWrapper.java(revision 177960)
 +++ ResponseWrapper.java(working copy)
 @@ -259,7 +259,7 @@
 
  if (open) {
  writer.write(/);
 -writer.close();
 +close();
  } else {
  writer.write(/);
  writer.write(name);
 
 
 -
 To unsubscribe, e-mail: [EMAIL PROTECTED]
 For additional commands, e-mail: [EMAIL PROTECTED]
 


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

[shale] Patch for org.apache.shale.remote.ResponseWrapper

[Shane Bryzak]

I hope this is the right place to post patches... anyway, I found a 
small bug in the ResponseWrapper class.  Patch as follows.


Index: ResponseWrapper.java
===
--- ResponseWrapper.java(revision 177960)
+++ ResponseWrapper.java(working copy)
@@ -259,7 +259,7 @@

if (open) {
writer.write(/);
-writer.close();
+close();
} else {
writer.write(/);
writer.write(name);


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