Seemantini,
Yes. I believe you are right. Thanks for the info. Regards, Ceki
At 14:33 15.08.2001 -0700, Seemantini Godbole wrote:
>Sorry I correct my earlier statement
>synchronized(this) should be changed to
>synchronized(SocketAppender.this) in my opinion.
>
>
>-----Original Message-----
>From: Seemantini Godbole
>Sent: Wednesday, August 15, 2001 4:27 PM
>To: 'LOG4J Users Mailing List'
>Subject: bug in SocketAppender?
>
>
>I was looking at some code in SocketAppender, particularly the following
>piece:
>I understand that the SocketAppender fires another thread, Connector,
>whenever it can not reach the remote server. The connector keeps trying
>every now and then to connect to the remote server and it sets the value of
>oos when it finally succeeds. It does a "synchronized" when it sets the
>value of oos. but according to me should the synchronized be on the outer
>class SocketAppender instead of connector. In essence, the statement
>synchronized(this) should be changed to
>synchronized(Connector.this) in my opinion.
>
>but, then again, I may be missing something...
>
>class Connector extends Thread {
>
> boolean interrupted = false;
>
> public
> void run() {
> Socket socket;
> while(!interrupted) {
> try {
> sleep(reconnectionDelay);
> LogLog.debug("Attempting connection to "+address.getHostName());
> socket = new Socket(address, port);
>
>/********The is the statement I am talking about **********/
> synchronized(this) {
> oos = new ObjectOutputStream(socket.getOutputStream());
> connector = null;
> break;
> }
> }
> catch(InterruptedException e) {
> LogLog.debug("Connector interrupted. Leaving loop.");
> return;
> }
>
>Any thoughts?
>
>---------------------------------------------------------------------
>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]
--
Ceki Gülcü - http://qos.ch
---------------------------------------------------------------------
To unsubscribe, e-mail: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]