From: | Bruce Momjian <maillist(at)candle(dot)pha(dot)pa(dot)us> |
---|---|
To: | Massimo Dal Zotto <dz(at)cs(dot)unitn(dot)it> |
Cc: | PostgreSQL Hackers <hackers(at)postgreSQL(dot)org>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Subject: | Re: [HACKERS] new patch |
Date: | 1999-07-19 16:45:51 |
Message-ID: | 199907191645.MAA16281@candle.pha.pa.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Applied.
[Charset ISO-8859-1 unsupported, filtering to ASCII...]
> >
> > Massimo Dal Zotto <dz(at)cs(dot)unitn(dot)it> writes:
> > > + char *progname = rindex(argv[0], SEP_CHAR);
> > > + rl_readline_name = (progname ? progname : argv[0]);
> >
> > 1. rindex => strrchr, please.
> >
> > 2. Shouldn't that be (progname ? progname+1 : argv[0]) ?
> >
> > regards, tom lane
> >
>
> Yes, I had already added the +1, but for same obscure reason it worked
> also with the original code, so I didn't notice the bug in my first test.
> I suspect that the readline uses only the basename of the argument.
> Ok also for strrchr, this is the final patch:
>
> *** src/bin/psql/psql.c.orig Sat Jun 5 09:00:38 1999
> --- src/bin/psql/psql.c Mon Jun 14 09:04:15 1999
> ***************
> *** 2875,2880 ****
> --- 2875,2887 ----
> pqsignal(SIGINT, handle_sigint); /* control-C => cancel */
> #ifdef USE_READLINE
> settings.useReadline = 1;
> + {
> + /*
> + * Set the application name, used for parsing .inputrc -- dz
> + */
> + char *progname = strrchr(argv[0], SEP_CHAR);
> + rl_readline_name = (progname ? progname+1 : argv[0]);
> + }
> #endif
> }
> #ifdef PSQL_ALWAYS_GET_PASSWORDS
>
>
> --
> Massimo Dal Zotto
>
> +----------------------------------------------------------------------+
> | Massimo Dal Zotto email: dz(at)cs(dot)unitn(dot)it |
> | Via Marconi, 141 phone: ++39-0461534251 |
> | 38057 Pergine Valsugana (TN) www: http://www.cs.unitn.it/~dz/ |
> | Italy pgp: finger dz(at)tango(dot)cs(dot)unitn(dot)it |
> +----------------------------------------------------------------------+
>
--
Bruce Momjian | http://www.op.net/~candle
maillist(at)candle(dot)pha(dot)pa(dot)us | (610) 853-3000
+ If your life is a hard drive, | 830 Blythe Avenue
+ Christ can be your backup. | Drexel Hill, Pennsylvania 19026
From | Date | Subject | |
---|---|---|---|
Next Message | Thomas Lockhart | 1999-07-19 16:58:50 | Re: [HACKERS] CVS |
Previous Message | Ansley, Michael | 1999-07-19 16:27:21 | RE: [HACKERS] CVS |