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
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/amsn-devel
--
KaKaRoTo
_______________________________________________
Amsn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/amsn-devel
