Currently, contrib/oid2name doesn't bother to free() the memory that it
malloc()'s. This isn't too serious (because oid2name is a short-lived
utility, so the memory will soon be returned to the OS on process
termination), but I still think it's poor style.
This patch changes oid2name so that it allocates memory on the stack
where possible and free()s the remaining heap-allocated memory. The
patch also fixes a typo a comment and adds 'const' qualifiers to a few
'char *' function parameters.
Neil Conway <neilconway(at)rogers(dot)com>
PGP Key ID: DB3C29FC
pgsql-patches by date
|Next:||From: Tom Lane||Date: 2002-02-24 21:35:49|
|Subject: Re: ALTER TABLE OWNER: change indexes |
|Previous:||From: Neil Conway||Date: 2002-02-24 21:13:30|
|Subject: ALTER TABLE OWNER: change indexes|