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

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