develooper Front page | perl.perl5.porters | Postings from March 2013

[perl #116959] [PATCH] perldtrace.pod: fix example code

Thread Next
From:
James E Keenan via RT
Date:
March 1, 2013 00:23
Subject:
[perl #116959] [PATCH] perldtrace.pod: fix example code
Message ID:
rt-3.6.HEAD-28177-1362097411-345.116959-15-0@perl.org
On Wed Feb 27 00:55:39 2013, me@lelf.tk wrote:
> 
> This is a bug report for perl from me@lelf.tk,
> generated with the help of perlbug 1.39 running under perl 5.10.1.
> 
> From c729127dc9379e4dffc5b03bff4038fbb57a2a6c Mon Sep 17 00:00:00 2001
> From: Anton Nikishaev <me@lelf.lu>
> Date: Wed, 27 Feb 2013 12:01:07 +0400
> Subject: [PATCH] perldtrace.pod: fix example code
> MIME-Version: 1.0
> Content-Type: multipart/mixed; boundary="------------1.7.12.2"
> 
> This is a multi-part message in MIME format.
> --------------1.7.12.2
> Content-Type: text/plain; charset=UTF-8; format=fixed
> Content-Transfer-Encoding: 8bit
> 
> ---
>  pod/perldtrace.pod | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> 
> --------------1.7.12.2
> Content-Type: text/x-patch; name="0001-perldtrace.pod-fix-example-
>    code.patch"
> Content-Transfer-Encoding: 8bit
> Content-Disposition: attachment; filename="0001-perldtrace.pod-fix-
>    example-code.patch"
> 
> diff --git a/pod/perldtrace.pod b/pod/perldtrace.pod
> index 022a237..0aeb268 100644
> --- a/pod/perldtrace.pod
> +++ b/pod/perldtrace.pod
> @@ -74,7 +74,7 @@ I<caller> from a DTrace action.
> 
>      :*perl*::sub-entry {
>          printf("%s::%s entered at %s line %d\n",
> -               copyinstr(arg3), copyinstr(arg0), copyinstr(arg1),
>    arg0);
> +               copyinstr(arg3), copyinstr(arg0), copyinstr(arg1),
>    arg2);
>      }
> 
>  =item sub-return(SUBNAME, FILE, LINE, PACKAGE)
> @@ -86,7 +86,7 @@ from a DTrace action.
> 
>      :*perl*::sub-return {
>          printf("%s::%s returned at %s line %d\n",
> -               copyinstr(arg3), copyinstr(arg0), copyinstr(arg1),
>    arg0);
> +               copyinstr(arg3), copyinstr(arg0), copyinstr(arg1),
>    arg2);
>      }
> 
>  =item phase-change(NEWPHASE, OLDPHASE)
> 
> --------------1.7.12.2--
> 
> 

I'm not dtrace-knowledgeable, so I can't say whether what we have in the
current POD is wrong or, if it is, whether the patch is the correct fix.

And dtrace experts available?

Thank you very much.
Jim Keenan


---
via perlbug:  queue: perl5 status: new
https://rt.perl.org:443/rt3/Ticket/Display.html?id=116959

Thread Next


nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About