Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor fix to map docs #15581

Closed
p5pRT opened this issue Sep 1, 2016 · 5 comments
Closed

Minor fix to map docs #15581

p5pRT opened this issue Sep 1, 2016 · 5 comments

Comments

@p5pRT
Copy link

p5pRT commented Sep 1, 2016

Migrated from rt.perl.org#129169 (status was 'resolved')

Searchable as RT129169$

@p5pRT
Copy link
Author

p5pRT commented Sep 1, 2016

From dave@mag-sol.com

The documentation for "map" in perlfunc mentions "an array" where it
really means "a list". The attached patch fixes this.

Dave...

@p5pRT
Copy link
Author

p5pRT commented Sep 1, 2016

From dave@mag-sol.com

fix-map-doc.diff
diff --git a/pod/perlfunc.pod b/pod/perlfunc.pod
index 176b02c..58316c7 100644
--- a/pod/perlfunc.pod
+++ b/pod/perlfunc.pod
@@ -4006,8 +4006,8 @@ Note that L<C<$_>|perlvar/$_> is an alias to the list value, so it can
 be used to modify the elements of the LIST.  While this is useful and
 supported, it can cause bizarre results if the elements of LIST are not
 variables.  Using a regular C<foreach> loop for this purpose would be
-clearer in most cases.  See also L<C<grep>|/grep BLOCK LIST> for an
-array composed of those items of the original list for which the BLOCK
+clearer in most cases.  See also L<C<grep>|/grep BLOCK LIST> for a
+list composed of those items of the original list for which the BLOCK
 or EXPR evaluates to true.
 
 C<{> starts both hash references and blocks, so C<map { ...> could be either

@p5pRT
Copy link
Author

p5pRT commented Sep 1, 2016

From @jkeenan

On Thu Sep 01 11​:51​:04 2016, dave@​mag-sol.com wrote​:

The documentation for "map" in perlfunc mentions "an array" where it
really means "a list". The attached patch fixes this.

Dave...

Thanks for the patch. Pushed to blead in commit commit 7c280be.

I believe this documentation correction should be available for the next version of 5.24 as well, so I will link it to the appropriate RT.

--
James E Keenan (jkeenan@​cpan.org)

@p5pRT
Copy link
Author

p5pRT commented Sep 1, 2016

The RT System itself - Status changed from 'new' to 'open'

@p5pRT
Copy link
Author

p5pRT commented Sep 1, 2016

@jkeenan - Status changed from 'open' to 'resolved'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant