Re: Patch: Remove gcc dependency in definition of inline functions

Marko Kreen <markokr@gmail.com>

From: Marko Kreen <markokr@gmail.com>
To: James Mansion <james@mansionfamily.plus.com>
Cc: Bruce Momjian <bruce@momjian.us>, Peter Eisentraut <peter_e@gmx.net>, Tom Lane <tgl@sss.pgh.pa.us>, Kurt Harriman <harriman@acm.org>, pgsql-hackers@postgresql.org
Date: 2009-12-02T21:30:12Z
Lists: pgsql-hackers
On 12/2/09, James Mansion <james@mansionfamily.plus.com> wrote:
> Marko Kreen wrote:
>
> > Note - my proposal would be to get rid of HAVE_INLINE, which
> > means we are already using inline functions unconditionally
> > on platforms that matter (gcc).  Keeping duplicate code
> > for obsolete compilers is pointless.
> >
> >
>  Microsoft C doesn't matter?
>
>  I seem to remember that when the Win32 version became available it actually
> increased the
>  number of people trying postgres rather dramatically.  Did that count for
> nothing?

The "(gcc)" above meant the inline functions are already used with gcc.

I have no reason to think Microsoft's inlining works worse than gcc's.

IOW - if the compiler does not support 'static inline' we should fall back
to plain 'static' functions, instead maintaining duplicate macros.
Such compilers would take a efficiency hit, but as they are practically
non-existent they dont matter.

Microsoft C does support inline, so it would not be affected.

-- 
marko