On Fri, 2008-11-21 at 13:22 +0530, Pavan Deolasee wrote:
> 
> I think we should avoid the #define's like below which uses a local
> variable. I guess the same #define is used elsewhere in the code as
> well. If the code is rearranged or the variable name is changed, the
> code would break.

I use a #define because same value is used elsewhere in code.

> The use of malloc should also be avoided (unless the memory subsystem
> is not up yet; I haven't checked).

The malloc was part of the existing code, explained by comments.

-- 
 Simon Riggs           www.2ndQuadrant.com
 PostgreSQL Training, Services and Support


-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Reply via email to