Search Postgresql Archives

Re: [HACKERS] Request - repeat value of \pset title during \watch interations

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Sat, Mar 19, 2016 at 11:42 PM, Robert Haas <robertmhaas@xxxxxxxxx> wrote:
> On Thu, Mar 10, 2016 at 1:40 AM, David G. Johnston
> <david.g.johnston@xxxxxxxxx> 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

Attachment: psql_watch_title-v3.patch
Description: binary/octet-stream

-- 
Sent via pgsql-general mailing list (pgsql-general@xxxxxxxxxxxxxx)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-general

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Index of Archives]     [Postgresql Jobs]     [Postgresql Admin]     [Postgresql Performance]     [Linux Clusters]     [PHP Home]     [PHP on Windows]     [Kernel Newbies]     [PHP Classes]     [PHP Books]     [PHP Databases]     [Postgresql & PHP]     [Yosemite]
  Powered by Linux