On Fri, Sep 05, 2003 at 06:01:12PM +0200, Asger Kunuk Alstrup wrote:
> On Fri, 5 Sep 2003, Angus Leeming wrote:
>
> > support/textutils.h cannot currently be compiled stand-alone. To do so would
> > require a
> > #include "paragraph.h".
> >
> > inline
> > bool IsInsetChar(char c)
> > {
> > return (c == Paragraph::META_INSET);
> > }
> >
> > So, should I do that or should I move this function out-of-line?
>
> I would not be surprised if this turns up in profiles, if you outline it.
What about 'manual inlining', i.e. instead of
if (IsInsetChar(c))
write
if (c == Paragraph::META_INSET)
whenever it is needed?
I don't think the wrapper function adds any sensible information or
clarity in this particular case.
[Note that I said 'in this particular case']
Andre'
