Re: Oaf patch ...



On Tue, 17 Jul 2001, Darin Adler wrote:

> >  	sort_criteria = NULL;
> > +
> > +        for (j = 0; j < 5; j++) {
> > +                oaf_name_service_get (&ev);
> > +                if (ev._major == CORBA_NO_EXCEPTION)
> > +                        g_warning ("Error getting name service '%s'", ev.
> > _id);
> > +        }
> >
> >  	if (argc > 1) {
> >  		query = argv[1];
>
> If you're going to have this loop to get the name service 5 times, you
> should at least write a comment and explain why you're doing it.

Hmm, the loop seems broken period - the registration location code needs
to work the first time...?

-- Elliot
If a single monkey presses random keys on a single typewriter for a single
second, it will discover that it needs to be subscribed to gnome-gui-list
to post.





[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]