utils/misc: Simplify search of end of argv in save_ps_display_args()

Lists: pgsql-hackers
From: Alexander Kuleshov <kuleshovmail(at)gmail(dot)com>
To: Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: utils/misc: Simplify search of end of argv in save_ps_display_args()
Date: 2016-03-10 08:41:10
Message-ID: CANCZXo4bLiNMari5J5Wusr+9-LQjx17Lm+yLkFLa3tge78uAKg@mail.gmail.com
Views: Whole Thread | Raw Message | Download mbox | Resend email
Lists: pgsql-hackers

Hello,

Attached patch provides trivial simplification of the search of end of
the argv[] area by replacing for() loop with calculation based on
argc.

Attachment Content-Type Size
simplify-ps-status.patch text/x-patch 583 bytes

From: Alvaro Herrera <alvherre(at)2ndquadrant(dot)com>
To: Alexander Kuleshov <kuleshovmail(at)gmail(dot)com>
Cc: Pg Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: utils/misc: Simplify search of end of argv in save_ps_display_args()
Date: 2016-03-10 14:36:31
Message-ID: [email protected]
Views: Whole Thread | Raw Message | Download mbox | Resend email
Lists: pgsql-hackers

Alexander Kuleshov wrote:
> Hello,
>
> Attached patch provides trivial simplification of the search of end of
> the argv[] area by replacing for() loop with calculation based on
> argc.

Uhm. Doesn't this break the very same thing that the comment explains
it's doing?

--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services