This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
threads::shared: Don’t link to moving targets
authorFather Chrysostomos <sprout@cpan.org>
Sat, 18 Jan 2014 13:36:52 +0000 (05:36 -0800)
committerFather Chrysostomos <sprout@cpan.org>
Sat, 18 Jan 2014 13:54:11 +0000 (05:54 -0800)
The Scalar::Util documentation has changed, so the links are broken.
But we cannot just update the link targets, as threads::shared is
living a double life and may be installed along with an older
Scalar::Util.

dist/threads-shared/lib/threads/shared.pm
t/porting/known_pod_issues.dat

index 5a24a98..1337b58 100644 (file)
@@ -336,7 +336,7 @@ then the C<undef> substitution will be performed silently.
 
 C<is_shared> checks if the specified variable is shared or not.  If shared,
 returns the variable's internal ID (similar to
-L<refaddr()|Scalar::Util/"refaddr EXPR">).  Otherwise, returns C<undef>.
+C<refaddr()> (see L<Scalar::Util>).  Otherwise, returns C<undef>.
 
   if (is_shared($var)) {
       print("\$var is shared\n");
@@ -584,7 +584,7 @@ the error "lock can only be used on shared values" to occur when you attempt
 to C<< lock($hashref->{key}) >> or C<< lock($arrayref->[idx]) >> in another
 thread.
 
-Using L<refaddr()|Scalar::Util/"refaddr EXPR">) is unreliable for testing
+Using C<refaddr()> is unreliable for testing
 whether or not two shared references are equivalent (e.g., when testing for
 circular references).  Use L<is_shared()|/"is_shared VARIABLE">, instead:
 
@@ -623,8 +623,8 @@ Either of the following will work instead:
         ...
     }
 
-This module supports dual-valued variables created using L<dualvar() from
-Scalar::Util|Scalar::Util/"dualvar NUM, STRING">).  However, while C<$!> acts
+This module supports dual-valued variables created using C<dualvar()> from
+L<Scalar::Util>.  However, while C<$!> acts
 like a dualvar, it is implemented as a tied SV.  To propagate its value, use
 the follow construct, if needed:
 
index dff00dd..7861b69 100644 (file)
@@ -203,7 +203,6 @@ dist/module-corelist/lib/module/corelist/utils.pm   Verbatim line length including
 dist/selfloader/lib/selfloader.pm      Verbatim line length including indents exceeds 79 by    13
 dist/storable/storable.pm      Verbatim line length including indents exceeds 79 by    4
 dist/thread-queue/lib/thread/queue.pm  Verbatim line length including indents exceeds 79 by    4
-dist/threads-shared/lib/threads/shared.pm      Apparent broken link    3
 dist/threads/lib/threads.pm    Verbatim line length including indents exceeds 79 by    3
 dist/tie-file/lib/tie/file.pm  Verbatim line length including indents exceeds 79 by    3
 ext/devel-peek/peek.pm ? Should you be using L<...> instead of 2