Re: core segfaults (Yahoo?)

On Fri, 19 Mar 2004, Philip S Tellis wrote:

> Sometime on Mar 17, A. Craig West assembled some asciibets to say:
> 
> >    printf("Hi-ho, hi-ho, Yahoo here we go!\n");
> > +  /*yahoo_set_log_level(YAHOO_LOG_DEBUG);*/
> 
> ??

This is to make it easy to find the correct place to turn on logging...

> >        break;
> >      }
> > +    case YAHOO_LOGIN_DUPL:
> > +    {
> > +      printf("Yahoo forced offline by duplicate login\n");
> 
> do you really want a printf in the lib?  should be a sprintf or a LOG.

Oops, you are correct, this should either be removed or switched to a LOG...

> 
> >  void ext_yahoo_got_conf_invite(int id, char *who, char *room, char *msg, YList *members)
> > -{}
> > +{
> > +}
> 
> heh.

And this was to make it easier to add temporary logging statements.

-- 
Craig West         Ph: (416) 666-1645	|  It's not a bug,
[email protected]              	|  It's a feature...