Though I do appreciate comments such as this: /* Fetch the function body */ tmp = SysCacheGetAttr(PROCOID, func_tuple, num_pg_proc_prosrc, &isNull); Ne

Though I do appreciate comments such as this: /* Fetch the function body */ tmp = SysCacheGetAttr(PROCOID, func_tuple, num_pg_proc_prosrc, &isNull); Newcomers have be be quite familiar with the internals to immediately see that the line fetches the function body.