On 2013-02-04 14:55:56 -0300, Alvaro Herrera wrote: > Andres Freund wrote: > > On 2013-02-04 13:22:26 -0300, Alvaro Herrera wrote: > > > > +typedef struct RmgrDescData > > > +{ > > > + void (*rm_desc) (StringInfo buf, uint8 xl_info, char *rec); > > > +} RmgrDescData; > > > + > > > +extern const RmgrDescData RmgrDescTable[]; > > > + > > > > I think we would at least need to include rm_name here as well. > > Not rm_name (that would be duplicative), but a comment with the name of > each rmgr's name in its line should be sufficient.
Maybe I am misunderstanding what you want to do, but xlogdump prints the name of the rmgr and allows to filter by it, how would we do that without rm_name? Greetings, Andres Freund -- Andres Freund http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services -- Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers