[PATCH] perf: fix duplicated words in documentation and comments

Markus Heidelberg posted 1 patch 6 days, 12 hours ago
tools/perf/Documentation/perf-diff.txt | 2 +-
tools/perf/builtin-script.c            | 2 +-
tools/perf/util/hwmon_pmu.h            | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
[PATCH] perf: fix duplicated words in documentation and comments
Posted by Markus Heidelberg 6 days, 12 hours ago
- "the the"
- "in in"
- "a a"

Signed-off-by: Markus Heidelberg <m.heidelberg@cab.de>
---
 tools/perf/Documentation/perf-diff.txt | 2 +-
 tools/perf/builtin-script.c            | 2 +-
 tools/perf/util/hwmon_pmu.h            | 2 +-
 3 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/tools/perf/Documentation/perf-diff.txt b/tools/perf/Documentation/perf-diff.txt
index f3067a4af294..58efab72d2e5 100644
--- a/tools/perf/Documentation/perf-diff.txt
+++ b/tools/perf/Documentation/perf-diff.txt
@@ -285,7 +285,7 @@ If specified the 'Weighted diff' column is displayed with value 'd' computed as:
 
   - period being the hist entry period value
 
-  - WEIGHT-A/WEIGHT-B being user supplied weights in the the '-c' option
+  - WEIGHT-A/WEIGHT-B being user supplied weights in the '-c' option
     behind ':' separator like '-c wdiff:1,2'.
     - WEIGHT-A being the weight of the data file
     - WEIGHT-B being the weight of the baseline data file
diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c
index d9fbdcf72f25..f731b6de7f23 100644
--- a/tools/perf/builtin-script.c
+++ b/tools/perf/builtin-script.c
@@ -223,7 +223,7 @@ enum {
 	OUTPUT_TYPE_MAX
 };
 
-// We need to refactor the evsel->priv use in in 'perf script' to allow for
+// We need to refactor the evsel->priv use in 'perf script' to allow for
 // using that area, that is being used only in some cases.
 #define OUTPUT_TYPE_UNSET -1
 
diff --git a/tools/perf/util/hwmon_pmu.h b/tools/perf/util/hwmon_pmu.h
index dc711b289ff5..d1e403c8b70b 100644
--- a/tools/perf/util/hwmon_pmu.h
+++ b/tools/perf/util/hwmon_pmu.h
@@ -37,7 +37,7 @@ enum hwmon_type {
 /**
  * enum hwmon_item:
  *
- * Similar to enum hwmon_type but describes the item part of a a sysfs filename.
+ * Similar to enum hwmon_type but describes the item part of a sysfs filename.
  *
  * This enum is exposed for testing.
  */
-- 
2.43.0
Re: [PATCH] perf: fix duplicated words in documentation and comments
Posted by Ian Rogers 6 days, 8 hours ago
On Thu, Sep 25, 2025 at 4:26 AM Markus Heidelberg <m.heidelberg@cab.de> wrote:
>
> - "the the"
> - "in in"
> - "a a"
>
> Signed-off-by: Markus Heidelberg <m.heidelberg@cab.de>

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

Thanks,
Ian

> ---
>  tools/perf/Documentation/perf-diff.txt | 2 +-
>  tools/perf/builtin-script.c            | 2 +-
>  tools/perf/util/hwmon_pmu.h            | 2 +-
>  3 files changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/tools/perf/Documentation/perf-diff.txt b/tools/perf/Documentation/perf-diff.txt
> index f3067a4af294..58efab72d2e5 100644
> --- a/tools/perf/Documentation/perf-diff.txt
> +++ b/tools/perf/Documentation/perf-diff.txt
> @@ -285,7 +285,7 @@ If specified the 'Weighted diff' column is displayed with value 'd' computed as:
>
>    - period being the hist entry period value
>
> -  - WEIGHT-A/WEIGHT-B being user supplied weights in the the '-c' option
> +  - WEIGHT-A/WEIGHT-B being user supplied weights in the '-c' option
>      behind ':' separator like '-c wdiff:1,2'.
>      - WEIGHT-A being the weight of the data file
>      - WEIGHT-B being the weight of the baseline data file
> diff --git a/tools/perf/builtin-script.c b/tools/perf/builtin-script.c
> index d9fbdcf72f25..f731b6de7f23 100644
> --- a/tools/perf/builtin-script.c
> +++ b/tools/perf/builtin-script.c
> @@ -223,7 +223,7 @@ enum {
>         OUTPUT_TYPE_MAX
>  };
>
> -// We need to refactor the evsel->priv use in in 'perf script' to allow for
> +// We need to refactor the evsel->priv use in 'perf script' to allow for
>  // using that area, that is being used only in some cases.
>  #define OUTPUT_TYPE_UNSET -1
>
> diff --git a/tools/perf/util/hwmon_pmu.h b/tools/perf/util/hwmon_pmu.h
> index dc711b289ff5..d1e403c8b70b 100644
> --- a/tools/perf/util/hwmon_pmu.h
> +++ b/tools/perf/util/hwmon_pmu.h
> @@ -37,7 +37,7 @@ enum hwmon_type {
>  /**
>   * enum hwmon_item:
>   *
> - * Similar to enum hwmon_type but describes the item part of a a sysfs filename.
> + * Similar to enum hwmon_type but describes the item part of a sysfs filename.
>   *
>   * This enum is exposed for testing.
>   */
> --
> 2.43.0
>
Re: [PATCH] perf: fix duplicated words in documentation and comments
Posted by Arnaldo Carvalho de Melo 10 hours ago
On Thu, Sep 25, 2025 at 08:18:58AM -0700, Ian Rogers wrote:
> On Thu, Sep 25, 2025 at 4:26 AM Markus Heidelberg <m.heidelberg@cab.de> wrote:
> >
> > - "the the"
> > - "in in"
> > - "a a"
> >
> > Signed-off-by: Markus Heidelberg <m.heidelberg@cab.de>
> 
> Reviewed-by: Ian Rogers <irogers@google.com>

Thanks, applied to perf-tools-next,

- Arnaldo