On 25/02/2011 09:57, Simone Tripodi wrote:
on a side note: Francesco, did you receive the account activation?
It's been a while now and looks like it's taking time...
please let me know!

Hi Simone,
unfortunately nothing has arrived so far :-(

On Tue, Feb 22, 2011 at 3:15 PM, Reinhard Pötz<reinh...@apache.org>  wrote:
On 02/21/2011 11:32 AM, Francesco Chicchiriccò wrote:
On 21/feb/2011, at 10.56, Simone Tripodi wrote:

Hi all, I'd tend agree with Reinhard if the CloseShield
functionality is not simple (and I mean very simple, almost silly)
to replicate into our module. I'm sure the CloseShield in the IO
takes care of more general PrintStream use cases rather then just
the sysout, so I'm worried that the proposed patch is not
enough...
For the sake of clarity: I've proposed that naive patch mainly
because I think that Log*Transformers are there to be used only for
dev purpose, where System.out or FileOutpuStream are the only viable
candidates.

Anyway, I've taken a quick look to
http://s.apache.org/commons-io-close-shield-outputstream and its
parent class http://bit.ly/h7AolY: it seems to me that it would be
quite easy to embed these two classes in order to have a CloseShield
functionality in cocoon3-sax.

If you think that it could be useful to have such functionality there
(also for usage by other classes than just Log*Transformers), please
let me know.
I think the patch with the simple approach is good enough in this case
because the class encapsulates the usage of the output stream completely.
(It would be different if an output stream was passed to the transformer
...)

Since you get commit access to the repository soon, you can apply the patch
yourself ;-) Congratulations BTW!

--
Reinhard Pötz         Founder&  Managing Director, Indoqa and Deepsearch
                        http://www.indoqa.com/people/reinhard-poetz.html

Member of the Apache Software Foundation
Apache Cocoon Committer, PMC member                  reinh...@apache.org
________________________________________________________________________

      Furthermore, I think Oracle has to honor the JSPA agreement.
    http://s.apache.org/JCPIsDead       http://s.apache.org/tck-trap

--
#########################################

Dott. Francesco Chicchiriccò
Amministratore unico
Tel +393290573276

Tirasa S.r.l.
Via XXXX Pescara
Tel / FAX +39085000000
http://www.tirasa.net

"To Iterate is Human, to Recurse, Divine"
(James O. Coplien, Bell Labs)

#########################################

Reply via email to