readded the pause problem...
don't forget to revert back your changes... and take a look at  
ReadFromSock, here :

                        default
                        {
                                status_log "option $state of socket $sock : 
[getObjOption $sock state]  
not defined.. receiving data [nbgets $sock]... closing \n" red
                                setObjOption $sock state "END"
                                catch { close $sock }
                        }

which means, if we receive the reflector's ACK, we'll go through all  
switch cases, won't find TSP_PAUSED, so it goes into default, which sets  
the state as END, closes the socket, and basta... so next time we 'resume'  
it, we won't be able to send on a closed socket... the readfromsock and  
writetosock proc will never get called because the $sock is closed...
which means that, first, you need to add a case "TSP_PAUSED", that will  
read the ACK from the reflector... secondly, test to see if we do receive  
it when sending a paused frame, and if it's the same length we need to  
[read ] and if it contains the same expected data. Also, if the ack is  
always received 4 seconds after sending the paused cam, it means we can  
remove our "after 4000 "fileevent writable..." " and make it writable only  
at the ACK... if the ack is sent in less than 4secs, then leave it the way  
it is...
Also, if we noticed, the socket gets closed but noone knows it got  
canceled, simply, because it never got canceled... the setobjoption stae  
END should have made it call the "END" branch of the switch, but only on  
the next ReadFromSock, which means never, since we closed the sock...
so I suggest you move the "END" code to the 'default' (where it calls  
CamCanceled), and make "END" have a '-' as body...
this will make sure that in the same situation, the socket won't only get  
closed, but will also get canceled, so you won't be chasing a wrong bug.
Make sure your status log talks about the option not being defined...

p.s.: re-reminding you to not forget to revert back the wrong change as  
it's really important!!!!

KKRT

On Fri, 02 Jun 2006 15:17:36 -0400, Vivia Nikolaidou <[EMAIL PROTECTED]>  
wrote:

> Removed:
>
> - Pause webcam works but resume doesn't
>
> COMPLETE TODO LIST FOR 0.96
>
> - We can't send files to msn8 users.
> - maybe apply holy_wood patch for saving DPs in a per_email directories  
> in
> the cache.
> - Make amsn 0.96 use TLS 1.5
>
>
> _______________________________________________
> Amsn-devel mailing list
> Amsn-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/amsn-devel



-- 
KaKaRoTo


_______________________________________________
Amsn-devel mailing list
Amsn-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/amsn-devel

Reply via email to