Skip site navigation (1) Skip section navigation (2)

Re: enable jade warnings

From: Neil Conway <neilc(at)samurai(dot)com>
To: Peter Eisentraut <peter_e(at)gmx(dot)net>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: enable jade warnings
Date: 2004-02-29 17:14:54
Message-ID: (view raw, whole thread or download thread mbox)
Lists: pgsql-patches
Peter Eisentraut wrote:
> If that doesn't create any new warnings on the current (fixed) code, 
> then I agree with this.

(Right, it does not.)

> However, I'd like to see it factored more to 
> avoid repeating it for jade and nsgmls.  The warning flags are in fact 
> SP flags, not Jade flags; only Jade is built on top of SP.  So maybe 
> assign this to SPFLAGS and add that to both invocations.  (This is a 
> bit analogous to CPPFLAGS vs. CFLAGS.)

Attached is a patch that does this.


Attachment: jade-warnings-2.patch
Description: text/x-patch (1.3 KB)

In response to

pgsql-patches by date

Next:From: Andrew DunstanDate: 2004-02-29 18:09:45
Subject: log_line_info
Previous:From: Peter EisentrautDate: 2004-02-29 17:01:25
Subject: Re: enable jade warnings

Privacy Policy | About PostgreSQL
Copyright © 1996-2017 The PostgreSQL Global Development Group