Tom Lane wrote: > Marti Raudsepp <ma...@juffo.org> writes: > > I happened to notice that there are no less than 14 places in the code > > that check whether a relation is a system catalog and throwing the > > error "permission denied: "foo" is a system catalog" > > > The attached patch factors all of those into a single > > ForbidSystemTableMods() function. Is this considered an improvement? > > I'd argue not. The code bulk savings is minimal, and this change > would degrade the usefulness of the file/line number reporting that's > built into ereport().
Along these lines, I've sometimes thought that it could be useful to pass down file/line info from certain callers down to certain generic check subroutines such as the one being proposed here. (I can't recall any specific examples offhand.) Of course, doing it manually would be very tedious and error prone, but perhaps we could have something like a macro system that both sets up arguments in the called function, and sets up the values in the callee. -- Álvaro Herrera 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
