[ekg2-devel] Obsługa <paused> w chatstates

[Wiesław Ochmiński]

On Mon, Jan 21, 2008 at 02:43:08PM +0100, Jakub Zawadzki wrote:

> Moje uwagi tylko nt. kodu.
> 
> > Index: plugins/jabber/jabber_handlers.c
> > ===================================================================
> > RCS file: /home/cvs/ekg2/plugins/jabber/jabber_handlers.c,v
> > retrieving revision 1.154
> > diff -u -r1.154 jabber_handlers.c
> > --- plugins/jabber/jabber_handlers.c	20 Jan 2008 08:31:48 -0000	1.154
> > +++ plugins/jabber/jabber_handlers.c	21 Jan 2008 11:28:25 -0000
> > @@ -968,12 +969,13 @@
> >  	 *   2nd bit determines whether we've got chatstate-based update,
> >  	 *     if its' on, then jabber:x:event can't replace the state;
> >  	 *   3rd bit determines whether the <composing/> is on
> > +	 *   4th bit determines whether the <paused/> is on	(+wiechu)
> >  	 */
> > -		int  state	= (!nbody && (composing & 4) ? EKG_XSTATE_TYPING : 0);
> > +		int  state	= (!nbody && (composing & 12) ? (composing & 4) ? EKG_XSTATE_TYPING : EKG_XSTATE_PAUSED : 0);
> 
> Ja tego nie pisalem, ale tam nie powinno byc (!nbody && composing & 4) ?
> [zamiast composing & 4 ?] (analogicznie jak wczesniej)
> 
> Btw. Skad sie wziela ta 12?
> (gdb) call/t 12
> $1 = 1100
> 
> Chyba sie nie zgadza z komentarzem:
>  * 4th bit determines whether the <paused/> is on 

4 to <composing>, 8 to <paused>. Jeśli którykolwiek (4|8 czyli 12), to
ustawiamy ten, który z tych dwóch wystąpił.
A ten czwarty bit (8) jest zapalane w
+                               composing = 11;

Według mnie wszystko jest w porządku.

    Pozdrawiam,
	W.
-- 
 Wiesław Ochmiński, Polska Północna, Malbork.