Build question
Danny Backx
danny.backx@scarlet.be
Tue Sep 8 15:53:00 GMT 2009
On Mon, 2009-09-07 at 15:11 -0700, Joel Brobecker wrote:
> > > +static inline int _isalpha(int c)
> > > +{
> > > + if (c <= 'Z' && c >= 'A')
> > > + return TRUE;
> > > + if (c <= 'z' && c >= 'a')
> > > + return TRUE;
> > > + return FALSE;
> > > +}
> >
> > I'm not sure what The Powers That Be think about defining inline
> > functions in a header.
>
> As far as I am concerned, I would really like us to avoid them.
> In this case, is the author trying to achieve performance or
> to get a behavior that's independent from the locale?
They are slightly more readable than the equivalent macros.
I've converted my code to use macros already. I need to add code to it
(to set the option) before I submit a new version.
Danny
--
Danny Backx ; danny.backx - at - scarlet.be ; http://danny.backx.info
More information about the Gdb-patches
mailing list