Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: Perl: porters

[PATCH] fix (m)XPUSH* usage in perlxstut

 

 

Perl porters RSS feed   Index | Next | Previous | View Threaded


ss5 at renormalist

Jul 2, 2012, 7:27 AM

Post #1 of 2 (49 views)
Permalink
[PATCH] fix (m)XPUSH* usage in perlxstut

Hi!

I found that XPUSH in perlxstut does not work as introduced. I
investigated a bit and according to git history it seems mXPUSH* is the
suggested way of doing it, since about d82b684c (2004, Steve Hay).

Attached patch fixes perlxstut example code and prose.

I actually tried the broken and the fixed code to verify the fix really
works.

Disclaimer: I'm new in this area, so still I might be wrong.

Kind regards,
Steffen
--
Steffen Schwigon <ss5 [at] renormalist>
Dresden Perl Mongers <http://dresden-pm.org/>
Attachments: 0001-perlxstut.pod-fix-m-XPUSH-usage.patch (2.80 KB)


bulk88 at hotmail

Jul 15, 2012, 5:50 PM

Post #2 of 2 (40 views)
Permalink
RE: [PATCH] fix (m)XPUSH* usage in perlxstut [In reply to]

----------------------------------------
> From: ss5 [at] renormalist
> To: perl5-porters [at] perl
> Subject: [PATCH] fix (m)XPUSH* usage in perlxstut
> Date: Mon, 2 Jul 2012 15:27:11 +0100
>
> Hi!
>
> I found that XPUSH in perlxstut does not work as introduced. I
> investigated a bit and according to git history it seems mXPUSH* is the
> suggested way of doing it, since about d82b684c (2004, Steve Hay).
>
> Attached patch fixes perlxstut example code and prose.
>
> I actually tried the broken and the fixed code to verify the fix really
> works.
>
> Disclaimer: I'm new in this area, so still I might be wrong.
>
> Kind regards,
> Steffen
> --
> Steffen Schwigon <ss5 [at] renormalist>
> Dresden Perl Mongers <http://dresden-pm.org/>
All those *XPUSH*es are terribly large machine code. One EXTEND please.

Perl porters RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.