Re: Fix for Extra Parenthesis in pgbench progress message

From: Nathan Bossart <nathandbossart(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Andres Freund <andres(at)anarazel(dot)de>, Fujii Masao <masao(dot)fujii(at)oss(dot)nttdata(dot)com>, Tatsuo Ishii <ishii(at)postgresql(dot)org>, pgsql-release(at)lists(dot)postgresql(dot)org, pgsql-hackers(at)postgresql(dot)org
Subject: Re: Fix for Extra Parenthesis in pgbench progress message
Date: 2025-02-07 18:13:37
Message-ID: Z6ZNUejz0L0pxQY4@nathan
Views: Whole Thread | Raw Message | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On Fri, Feb 07, 2025 at 01:10:04PM -0500, Tom Lane wrote:
> Nathan Bossart <nathandbossart(at)gmail(dot)com> writes:
>> Oops, I was already taking a look at this. I figured it'd just be
>> something like the following, although maybe there's a more elegant way.
>
> Well, the stuff with prev_chars really ought to be skipped as well.
> (Yeah, it's probably a no-op, but readers shouldn't have to figure
> that out.)
>
> My thought was that duplicating the logic isn't so bad, as attached.

WFM!

--
nathan

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2025-02-07 18:14:57 Re: Fix for Extra Parenthesis in pgbench progress message
Previous Message Andres Freund 2025-02-07 18:11:16 Re: Fix for Extra Parenthesis in pgbench progress message