* [PATCH] perf script python: fix unintended underline
@ 2021-08-06 20:45 Stephen Brennan
2021-08-06 21:13 ` Paul A. Clarke
2021-08-09 19:55 ` Arnaldo Carvalho de Melo
0 siblings, 2 replies; 5+ messages in thread
From: Stephen Brennan @ 2021-08-06 20:45 UTC (permalink / raw)
To: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Mark Rutland, Alexander Shishkin, Jiri Olsa, Namhyung Kim
Cc: Stephen Brennan, linux-perf-users, linux-kernel
The text ranging from "subsystem__event_name" to
"raw_syscalls__sys_enter()" is interpreted by asciidoc as a pair of
unconstrained text formatting markers. The result is that the manual
page displayed this text as underlined, and the HTML pages displayed
this text as italicized. Escape the first double-underscore to prevent
this.
https://docs.asciidoctor.org/asciidoc/latest/syntax-quick-reference/
Signed-off-by: Stephen Brennan <stephen.s.brennan@oracle.com>
---
tools/perf/Documentation/perf-script-python.txt | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/perf/Documentation/perf-script-python.txt b/tools/perf/Documentation/perf-script-python.txt
index 5e43cfa5ea1e..0250dc61cf98 100644
--- a/tools/perf/Documentation/perf-script-python.txt
+++ b/tools/perf/Documentation/perf-script-python.txt
@@ -167,7 +167,7 @@ below).
Following those are the 'event handler' functions generated one for
every event in the 'perf record' output. The handler functions take
-the form subsystem__event_name, and contain named parameters, one for
+the form subsystem\__event_name, and contain named parameters, one for
each field in the event; in this case, there's only one event,
raw_syscalls__sys_enter(). (see the EVENT HANDLERS section below for
more info on event handlers).
--
2.30.2
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH] perf script python: fix unintended underline
2021-08-06 20:45 [PATCH] perf script python: fix unintended underline Stephen Brennan
@ 2021-08-06 21:13 ` Paul A. Clarke
2021-08-06 21:28 ` Stephen Brennan
2021-08-09 19:55 ` Arnaldo Carvalho de Melo
1 sibling, 1 reply; 5+ messages in thread
From: Paul A. Clarke @ 2021-08-06 21:13 UTC (permalink / raw)
To: Stephen Brennan
Cc: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Mark Rutland, Alexander Shishkin, Jiri Olsa, Namhyung Kim,
linux-perf-users, linux-kernel
On Fri, Aug 06, 2021 at 01:45:01PM -0700, Stephen Brennan wrote:
> The text ranging from "subsystem__event_name" to
> "raw_syscalls__sys_enter()" is interpreted by asciidoc as a pair of
> unconstrained text formatting markers. The result is that the manual
> page displayed this text as underlined, and the HTML pages displayed
> this text as italicized. Escape the first double-underscore to prevent
> this.
I think it would be better to escape the second double-underscore as well,
to prevent the same problem recurring with future changes.
> diff --git a/tools/perf/Documentation/perf-script-python.txt b/tools/perf/Documentation/perf-script-python.txt
> index 5e43cfa5ea1e..0250dc61cf98 100644
> --- a/tools/perf/Documentation/perf-script-python.txt
> +++ b/tools/perf/Documentation/perf-script-python.txt
> @@ -167,7 +167,7 @@ below).
>
> Following those are the 'event handler' functions generated one for
> every event in the 'perf record' output. The handler functions take
> -the form subsystem__event_name, and contain named parameters, one for
> +the form subsystem\__event_name, and contain named parameters, one for
> each field in the event; in this case, there's only one event,
> raw_syscalls__sys_enter(). (see the EVENT HANDLERS section below for
escape this ^ , too.
> more info on event handlers).
> --
PC
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] perf script python: fix unintended underline
2021-08-06 21:13 ` Paul A. Clarke
@ 2021-08-06 21:28 ` Stephen Brennan
2021-08-06 22:28 ` Paul A. Clarke
0 siblings, 1 reply; 5+ messages in thread
From: Stephen Brennan @ 2021-08-06 21:28 UTC (permalink / raw)
To: Paul A. Clarke
Cc: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Mark Rutland, Alexander Shishkin, Jiri Olsa, Namhyung Kim,
linux-perf-users, linux-kernel
"Paul A. Clarke" <pc@us.ibm.com> writes:
> On Fri, Aug 06, 2021 at 01:45:01PM -0700, Stephen Brennan wrote:
>> The text ranging from "subsystem__event_name" to
>> "raw_syscalls__sys_enter()" is interpreted by asciidoc as a pair of
>> unconstrained text formatting markers. The result is that the manual
>> page displayed this text as underlined, and the HTML pages displayed
>> this text as italicized. Escape the first double-underscore to prevent
>> this.
>
> I think it would be better to escape the second double-underscore as well,
> to prevent the same problem recurring with future changes.
>
>> diff --git a/tools/perf/Documentation/perf-script-python.txt b/tools/perf/Documentation/perf-script-python.txt
>> index 5e43cfa5ea1e..0250dc61cf98 100644
>> --- a/tools/perf/Documentation/perf-script-python.txt
>> +++ b/tools/perf/Documentation/perf-script-python.txt
>> @@ -167,7 +167,7 @@ below).
>>
>> Following those are the 'event handler' functions generated one for
>> every event in the 'perf record' output. The handler functions take
>> -the form subsystem__event_name, and contain named parameters, one for
>> +the form subsystem\__event_name, and contain named parameters, one for
>> each field in the event; in this case, there's only one event,
>> raw_syscalls__sys_enter(). (see the EVENT HANDLERS section below for
>
> escape this ^ , too.
I've tried escaping a few combinations of these four underscores.
\__ __ - escaping the first but not the second (as this patch)
produces correct output
\_\_ \_\_ - escaping all underscores results in no underlines, but the
manual page shows "raw_syscalls\_\_sys_enter()" in its
output.
\_\_ __ - escaping the first two results in no underlines, but the
manual page shows "subsystem\_\_event_name"
\__ \__ - escaping the first of each double-underscore results in no
underlines, but the manual page shows
"raw_syscalls\__sys_enter()"
It seems that asciidoc only allows the first in a potential pair to be
escaped? I'll be the first to admit, I know nothing about asciidoc, so I
may have missed something here.
Stephen
>
>> more info on event handlers).
>> --
>
> PC
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH] perf script python: fix unintended underline
2021-08-06 21:28 ` Stephen Brennan
@ 2021-08-06 22:28 ` Paul A. Clarke
0 siblings, 0 replies; 5+ messages in thread
From: Paul A. Clarke @ 2021-08-06 22:28 UTC (permalink / raw)
To: Stephen Brennan
Cc: Peter Zijlstra, Ingo Molnar, Arnaldo Carvalho de Melo,
Mark Rutland, Alexander Shishkin, Jiri Olsa, Namhyung Kim,
linux-perf-users, linux-kernel
On Fri, Aug 06, 2021 at 02:28:13PM -0700, Stephen Brennan wrote:
> "Paul A. Clarke" <pc@us.ibm.com> writes:
> > On Fri, Aug 06, 2021 at 01:45:01PM -0700, Stephen Brennan wrote:
> >> The text ranging from "subsystem__event_name" to
> >> "raw_syscalls__sys_enter()" is interpreted by asciidoc as a pair of
> >> unconstrained text formatting markers. The result is that the manual
> >> page displayed this text as underlined, and the HTML pages displayed
> >> this text as italicized. Escape the first double-underscore to prevent
> >> this.
> >
> > I think it would be better to escape the second double-underscore as well,
> > to prevent the same problem recurring with future changes.
> >
> >> diff --git a/tools/perf/Documentation/perf-script-python.txt b/tools/perf/Documentation/perf-script-python.txt
> >> index 5e43cfa5ea1e..0250dc61cf98 100644
> >> --- a/tools/perf/Documentation/perf-script-python.txt
> >> +++ b/tools/perf/Documentation/perf-script-python.txt
> >> @@ -167,7 +167,7 @@ below).
> >>
> >> Following those are the 'event handler' functions generated one for
> >> every event in the 'perf record' output. The handler functions take
> >> -the form subsystem__event_name, and contain named parameters, one for
> >> +the form subsystem\__event_name, and contain named parameters, one for
> >> each field in the event; in this case, there's only one event,
> >> raw_syscalls__sys_enter(). (see the EVENT HANDLERS section below for
> >
> > escape this ^ , too.
>
> I've tried escaping a few combinations of these four underscores.
>
> \__ __ - escaping the first but not the second (as this patch)
> produces correct output
> \_\_ \_\_ - escaping all underscores results in no underlines, but the
> manual page shows "raw_syscalls\_\_sys_enter()" in its
> output.
> \_\_ __ - escaping the first two results in no underlines, but the
> manual page shows "subsystem\_\_event_name"
> \__ \__ - escaping the first of each double-underscore results in no
> underlines, but the manual page shows
> "raw_syscalls\__sys_enter()"
>
> It seems that asciidoc only allows the first in a potential pair to be
> escaped? I'll be the first to admit, I know nothing about asciidoc, so I
> may have missed something here.
Ugh. Thanks for trying. I thought it would be easy, but that probably shows
I know even less about asciidoc. :-/
PC
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH] perf script python: fix unintended underline
2021-08-06 20:45 [PATCH] perf script python: fix unintended underline Stephen Brennan
2021-08-06 21:13 ` Paul A. Clarke
@ 2021-08-09 19:55 ` Arnaldo Carvalho de Melo
1 sibling, 0 replies; 5+ messages in thread
From: Arnaldo Carvalho de Melo @ 2021-08-09 19:55 UTC (permalink / raw)
To: Stephen Brennan
Cc: Peter Zijlstra, Ingo Molnar, Mark Rutland, Alexander Shishkin,
Jiri Olsa, Namhyung Kim, linux-perf-users, linux-kernel
Em Fri, Aug 06, 2021 at 01:45:01PM -0700, Stephen Brennan escreveu:
> The text ranging from "subsystem__event_name" to
> "raw_syscalls__sys_enter()" is interpreted by asciidoc as a pair of
> unconstrained text formatting markers. The result is that the manual
> page displayed this text as underlined, and the HTML pages displayed
> this text as italicized. Escape the first double-underscore to prevent
> this.
>
> https://docs.asciidoctor.org/asciidoc/latest/syntax-quick-reference/
Thanks, applied.
- Arnaldo
> Signed-off-by: Stephen Brennan <stephen.s.brennan@oracle.com>
> ---
> tools/perf/Documentation/perf-script-python.txt | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/tools/perf/Documentation/perf-script-python.txt b/tools/perf/Documentation/perf-script-python.txt
> index 5e43cfa5ea1e..0250dc61cf98 100644
> --- a/tools/perf/Documentation/perf-script-python.txt
> +++ b/tools/perf/Documentation/perf-script-python.txt
> @@ -167,7 +167,7 @@ below).
>
> Following those are the 'event handler' functions generated one for
> every event in the 'perf record' output. The handler functions take
> -the form subsystem__event_name, and contain named parameters, one for
> +the form subsystem\__event_name, and contain named parameters, one for
> each field in the event; in this case, there's only one event,
> raw_syscalls__sys_enter(). (see the EVENT HANDLERS section below for
> more info on event handlers).
> --
> 2.30.2
>
--
- Arnaldo
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2021-08-09 19:55 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-06 20:45 [PATCH] perf script python: fix unintended underline Stephen Brennan
2021-08-06 21:13 ` Paul A. Clarke
2021-08-06 21:28 ` Stephen Brennan
2021-08-06 22:28 ` Paul A. Clarke
2021-08-09 19:55 ` Arnaldo Carvalho de Melo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).