pg.ddx.io  pgsql-interfaces@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Bruce Momjian <pgman@candle.pha.pa.us>
To: Peter Eisentraut <peter_e@gmx.net>
Cc: Tom Lane <tgl@sss.pgh.pa.us>
Cc: Dan Lyke <danlyke@flutterby.com>
Cc: pgsql-interfaces@postgresql.org
Subject: Re: Double newline bug with DBD::Pg: Where best to fix?
Date: Wed, 28 Feb 2001 12:42:05 -0500 (EST)
Message-ID: <200102281742.MAA04577@candle.pha.pa.us> (raw)
In-Reply-To: <Pine.LNX.4.30.0102281807470.775-100000@peter.localdomain>

> Both of these whitespace-away-optimizing checks could probably be removed
> completely, but I'm not sure whether there isn't some case that relies on
> it.  I'm gonna stare at it for a few hours and then check it in.

Not sure why the optimizations are there either, though I thought I put
them in there myself.

However, now that I look at the diff, the tests were there in your
initial 1.1 version too, I just moved them.  I see no problem removing
the tests, though at this late date, maybe the removal should happen in
7.2.  Not sure.

-- 
  Bruce Momjian                        |  http://candle.pha.pa.us
  pgman@candle.pha.pa.us               |  (610) 853-3000
  +  If your life is a hard drive,     |  830 Blythe Avenue
  +  Christ can be your backup.        |  Drexel Hill, Pennsylvania 19026



view thread (10+ messages)  latest in thread

Message-ID: <200102281742.MAA04577@candle.pha.pa.us>
Permalink:  ../200102281742.MAA04577@candle.pha.pa.us/
Also on:    postgresql.org/message-id/200102281742.MAA04577@candle.pha.pa.us

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-interfaces@postgresql.org
  Cc: pgman@candle.pha.pa.us, peter_e@gmx.net, tgl@sss.pgh.pa.us, danlyke@flutterby.com
  Subject: Re: Double newline bug with DBD::Pg: Where best to fix?
  In-Reply-To: <200102281742.MAA04577@candle.pha.pa.us>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox