[ekg2-devel] Ludzie IRCa. (people.c)

[Jakub Zawadzki]

On Tue, Feb 05, 2008 at 11:20:55PM +0100, GiM wrote:
> Wiesław Ochmiński in message 'Re: [ekg2-devel] Ludzie IRCa. (people.c)' wrote:
> > On Tue, Feb 05, 2008 at 07:17:17PM +0100, Wiesław Ochmiński wrote:
> 
> To mój kod więc się do niego ustosunkuję
> 
> > Index: plugins/irc/people.c
> > ===================================================================
> > --- plugins/irc/people.c	(wersja 3739)
> > +++ plugins/irc/people.c	(kopia robocza)
> >  
> > @@ -165,6 +166,9 @@
> >  	/*	debug("+userlisty %d, ", mode); */
> >  		ulist = userlist_add_u(&(w->userlist), ircnick, nick);
> >  		irccol = irc_color_in_contacts(modes, mode, ulist);
> > +		char *__session = xstrdup(session_uid_get(s));
> > +		query_emit_id(NULL, USERLIST_CHANGED, &__session, &(ulist->uid));
> > +		xfree(__session);
> >  	}
> >  

To IMHO nie powinno byc w irc_add_person_int() tylko w irc_add_person() oraz irc_add_people()
[po tej petli z irc_add_person_int()]

> Po drugie tego tam nie ma, z określonych powodów i póki co raczej nie
> będzie, a jeśli to nie w tej formie.

GiM, to wymysl jakis sposob na informowanie UI, ze sie zmienila userlista, i ze trzeba ja redrawnac.

Robienie przez eventy, jest proper ekg2-way. Moze inny event, niekoniecznie USERLIST_CHANGED
(zeby nie trzeba bylo przekazywac wartosci, tylko zeby ui odswiezylo userliste)

Ale ogolnie, to _trzeba_ to zrobic, niekoniecznie teraz, mozna po sesji.

Pozdrawiam.