Re: split rm_name and rm_desc out of rmgr.c

On 2/5/13 3:47 PM, Alvaro Herrera wrote:
> Tom Lane wrote:
>> Alvaro Herrera <alvherre@2ndquadrant.com> writes:
> 
>>> The
>>> approach in the second patch is to turn these into "extern const RmgrId"
>>> instead, and use a second inclusion of rmgrlist.h in rmgr.c that assigns
>>> them the values as consts.
>>
>> ... but I don't especially like that implementation, as it will result
>> in nonzero code bloat and runtime cost due to replacing all those
>> constants with global-variable references.  Couldn't you instead set it
>> up as an enum definition?
> 
> That seems to work.  I would like to have some way of specifying that
> the enum members should be of type RmgrId, but I don't think there's any
> way to do that.
> 
> Patch attached.

This has broken cpluspluscheck:

./src/include/access/rmgrlist.h:28:8: error: expected constructor, destructor, or type conversion before '(' token