Re: [AD] get_config_string return value

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


> Thanks. I've committed it, with some reformatting.

If I were a purist, I would say that:

if (ustricmp(p->name, name) == 0) {
   /* check if entry really exists */
   if (p->data && (ustrlen(p->data) != 0))
      return p;
   else
      return NULL;
}

is better.

Moreover, the test line 695 is now useless and get_config_string() can't
return 'empty string' for unhooked sections any longer.

--
Eric Botcazou
ebotcazou@xxxxxxxxxx



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