Re: Why does load_external_function() return PGFunction? - Mailing list pgsql-hackers

From Andres Freund
Subject Re: Why does load_external_function() return PGFunction?
Date
Msg-id 20180324203320.fsdm2qsibfcvgpsb@alap3.anarazel.de
Whole thread Raw
In response to Re: Why does load_external_function() return PGFunction?  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Why does load_external_function() return PGFunction?
List pgsql-hackers
On 2018-02-06 15:43:29 -0500, Tom Lane wrote:
> Andres Freund <andres@anarazel.de> writes:
> > We've several callers to load_external_function() that do not use the
> > returned value as a PGFunction. I'd vote for changing the return type to
> > void * and have fmgr.c cast it to PGFunction after verifying the
> > function's magic.
> 
> void* isn't necessarily compatible with function pointers --- there are
> platforms where they're physically different widths, though possibly
> you'd never get PG to run on such hardware anyway.
> 
> I'd be OK with declaring it as a more generic function pointer type,
> perhaps "void (*funcptr) ()".
> 
> However, given that a cast is going to be necessary anyway, it seems
> like this is mostly useless churn...

I don't think it really changes the need, but it's worthwhile to note
that gcc-8 warns about this now:
/home/andres/src/postgresql/src/backend/postmaster/bgworker.c: In function ‘LookupBackgroundWorkerFunction’:
/home/andres/src/postgresql/src/backend/postmaster/bgworker.c:1246:9: warning: cast between incompatible function types
from‘PGFunction’ {aka ‘long unsigned int (*)(struct FunctionCallInfoData *)’} to ‘void (*)(Datum)’ {aka ‘void (*)(long
unsignedint)’} [-Wcast-function-type]
 
  return (bgworker_main_type)

Greetings,

Andres Freund


pgsql-hackers by date:

Previous
From: Alexander Korotkov
Date:
Subject: Re: WIP: Covering + unique indexes.
Next
From: Andres Freund
Date:
Subject: Re: Undesirable entries in typedefs list