Re: PL/R regression on windows, but not linux with master. - Mailing list pgsql-hackers

From Tom Lane
Subject Re: PL/R regression on windows, but not linux with master.
Date
Msg-id 2667316.1618150403@sss.pgh.pa.us
Whole thread Raw
In response to Re: PL/R regression on windows, but not linux with master.  (Andrew Dunstan <andrew@dunslane.net>)
Responses Re: PL/R regression on windows, but not linux with master.
List pgsql-hackers
Andrew Dunstan <andrew@dunslane.net> writes:
>> Well, plr.h does this:
>> 
>> #define WARNING        19
>> #define ERROR        20

> The coding pattern in plr.h looks quite breakable. Instead of hard
> coding values like this they should save the value from the postgres
> headers in another variable before undefining it and then restore that
> value after inclusion of the R headers.

Indeed.  elog.h already provides a "PGERROR" macro to use for restoring
the value of ERROR.  We have not heard of a need to do anything special
for WARNING though --- maybe that's R-specific?

            regards, tom lane



pgsql-hackers by date:

Previous
From: Zhihong Yu
Date:
Subject: Re: Add header support to text format and matching feature
Next
From: Rémi Lapeyre
Date:
Subject: Re: Add header support to text format and matching feature