Re: [AD] Allegro 5 new config routines, alpha 2

[ Thread Index | Date Index | More lists.liballeg.org/allegro-developers Archives ]


On vie, feb 08, 2002 at 03:40:36 +1100, Peter Wang wrote:
> On 2002-02-07, Angelo Mottola <a.mottola@xxxxxxxxxx> wrote:
> >    /* Avoid collisions (if any) */
> >    while ((strcmp(entry->name, name)) && (entry->next))
> >       entry = entry->next;
> >    if (strcmp(entry->name, name))
> >       return NULL;
> 
> Just quickly, there seems to be lots of these.  strcmp'ing might be
> unnecessarily called twice.
> 
>      while (entry && strcmp(entry->name, name))
> 	entry = entry->next;
>      if (!entry)
> 	return NULL;
> 
> .. or something like that might be better.

Here a patch with:

  - Peter Wang recomendation;
  - The `float' type;
  - Unification of all types in an `union';
  - Some wish-list;

-- 
David A. Capello
dacap@xxxxxxxxxx
http://www.davidcapello.com.ar

Attachment: config.diff.gz
Description: Binary data



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/