Re: [PATCH v2] perf trace: avoid garbage when not printing a trace event's arguments
From: Arnaldo Carvalho de Melo
Date: Fri Nov 01 2024 - 20:38:16 EST
On Fri, Nov 01, 2024 at 02:41:46PM -0700, Benjamin Peterson wrote:
> On Fri, Nov 1, 2024 at 2:30 PM Arnaldo Carvalho de Melo <acme@xxxxxxxxxx> wrote:
> > I haven't tested it yet, just in my mind :-)
> > The patch looks ok and seems to fix a real problem, my only concern, a
> > pet peeve, was that it, in addition to fixing a real problem, did an
> > unrelated change, the "Remove the return value...", that part looks like
> > a distraction, something that shouldn't be there.
> In my mind, it was related because both the bug and the odd return
> value arise from incorrect counting of the number of written bytes.
> Obviously, the fix is what I care about, so feel free to strip out the
> return value change on import, or I can resend.
The point is to try to focus on the fix, and do just what is needed for
that. Everything else ends up being a distraction.
- Arnaldo