On Sat, Mar 19, 2016 at 11:42 PM, Robert Haas <[email protected]> wrote: > On Thu, Mar 10, 2016 at 1:40 AM, David G. Johnston > <[email protected]> wrote: >> Adding -hackers for consideration in the Commitfest. > > I don't much like how this patch uses the arbitrary constant 50 in no > fewer than 5 locations. > > Also, it seems like we could arrange for head_title to be "" rather > than NULL when myopt.title is NULL. Then instead of this: > > + if (head_title) > + snprintf(title, strlen(myopt.title) + 50, > + _("Watch every %lds\t%s\n%s"), > + sleep, asctime(localtime(&timer)), head_title); > + else > + snprintf(title, 50, _("Watch every %lds\t%s"), > + sleep, asctime(localtime(&timer))); > > ...we could just the first branch of that if all the time.
OK, why not.
> if (res == -1)
> + {
> + pg_free(title);
> + pg_free(head_title);
> return false;
> + }
>
> Instead of repeating the cleanup code, how about making this break;
> then, change the return statement at the bottom of the function to
> return (res != -1).
OK.
And the patch attached gives the following output:
With title:
=# \watch 1
Watch every 1s Sun Mar 20 22:28:38 2016
popo
a
---
1
(1 row)
And without title:
Watch every 1s Sun Mar 20 22:29:31 2016
a
---
1
(1 row)
--
Michael
psql_watch_title-v3.patch
Description: binary/octet-stream
-- Sent via pgsql-hackers mailing list ([email protected]) To make changes to your subscription: http://www.postgresql.org/mailpref/pgsql-hackers
