glibc str functions const return fixes

[PATCH perf-next 0/4] glibc str functions const return fixes

Arnaldo Carvalho de Melo posted 4 patches 2 days, 3 hours ago

Download series mbox

tools/perf/builtin-diff.c       | 12 +++++-------
tools/perf/builtin-list.c       |  4 ++--
tools/perf/builtin-trace.c      | 11 +++++++----
tools/perf/jvmti/libjvmti.c     |  2 +-
tools/perf/tests/parse-events.c |  4 ++--
tools/perf/util/evlist.c        |  3 ++-
6 files changed, 19 insertions(+), 17 deletions(-)

Expand all Fold all

[PATCH perf-next 0/4] glibc str functions const return fixes

Posted by Arnaldo Carvalho de Melo 2 days, 3 hours ago

Hi,

	While building with Fedora 44 I noticed so interesting warnings
about touching const memory, fix some of them in this series,

- Arnaldo

Arnaldo Carvalho de Melo (4):
  perf list: Remove unused 'sep' variable
  perf diff: Constify strchr() return variables
  perf tools: Use const for variables receiving str{str,r?chr}() returns
  perf trace: Don't change const char strings

 tools/perf/builtin-diff.c       | 12 +++++-------
 tools/perf/builtin-list.c       |  4 ++--
 tools/perf/builtin-trace.c      | 11 +++++++----
 tools/perf/jvmti/libjvmti.c     |  2 +-
 tools/perf/tests/parse-events.c |  4 ++--
 tools/perf/util/evlist.c        |  3 ++-
 6 files changed, 19 insertions(+), 17 deletions(-)

-- 
2.52.0

Re: [PATCH perf-next 0/4] glibc str functions const return fixes

Posted by Ian Rogers 1 day, 20 hours ago

On Thu, Dec 11, 2025 at 2:18 PM Arnaldo Carvalho de Melo
<acme@kernel.org> wrote:
>
> Hi,
>
>         While building with Fedora 44 I noticed so interesting warnings
> about touching const memory, fix some of them in this series,
>
> - Arnaldo

For the series:

Reviewed-by: Ian Rogers <irogers@google.com>

Thanks,
Ian

>
> Arnaldo Carvalho de Melo (4):
>   perf list: Remove unused 'sep' variable
>   perf diff: Constify strchr() return variables
>   perf tools: Use const for variables receiving str{str,r?chr}() returns
>   perf trace: Don't change const char strings
>
>  tools/perf/builtin-diff.c       | 12 +++++-------
>  tools/perf/builtin-list.c       |  4 ++--
>  tools/perf/builtin-trace.c      | 11 +++++++----
>  tools/perf/jvmti/libjvmti.c     |  2 +-
>  tools/perf/tests/parse-events.c |  4 ++--
>  tools/perf/util/evlist.c        |  3 ++-
>  6 files changed, 19 insertions(+), 17 deletions(-)
>
> --
> 2.52.0
>