Re: SEP_CHAR - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: SEP_CHAR
Date
Msg-id 200105231548.f4NFmVV12616@candle.pha.pa.us
Whole thread Raw
In response to Re: SEP_CHAR  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: SEP_CHAR
Re: SEP_CHAR
List pgsql-hackers
> Peter Eisentraut <peter_e@gmx.net> writes:
> > Bruce Momjian writes:
> >> I see some use of SEP_CHAR for '/' in the source.  Do we want to use it
> >> consistenly, or not use it at all?  I can make the change.
> 
> > I vote for not at all, since there is not at all a use for it.
> 
> If it's not actually needed for the Cygwin port, I'd vote for taking it
> out.  It adds clutter to the sources, to no purpose.  Also, even if you
> fix it to be used consistently *every* place it should be, how long will
> it stay that way?  Seems practically unmaintainable to me.

I see it used by psql and pg_dump and I will leave them alone, assuming
/ is not hardeded elsewhere in that code.

Actually, it is:
           sprintf(psqlrc, "%s/.psqlrc", home);

so it seems NT handles '/' anyway.

--  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,
Pennsylvania19026
 


pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: Not released yet, but could someone take a quick peak ...
Next
From: Bruce Momjian
Date:
Subject: Re: SEP_CHAR