Skip Menu |
 

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Perl-Critic CPAN distribution.

Report information
The Basics
Id: 59111
Status: new
Priority: 0/
Queue: Perl-Critic

People
Owner: Nobody in particular
Requestors: user42 [...] zip.com.au
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



From gg [...] zip.com.au Tue Jul 6 04: 20:04 2010
MIME-Version: 1.0
X-Spam-Status: No, score=-10.57 tagged_above=-99.9 required=10 tests=[AWL=0.029, BAYES_00=-2.599, RCVD_IN_DNSWL_HI=-8] autolearn=ham
X-Spam-Flag: NO
content-type: text/plain; charset="utf-8"
Message-ID: <871vbhlaoe.fsf [...] blah.blah>
X-Virus-Scanned: Debian amavisd-new at bestpractical.com
X-Spam-Score: -10.57
Received: from localhost (localhost [127.0.0.1]) by hipster.bestpractical.com (Postfix) with ESMTP id C99A161E00A for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Mon, 5 Jul 2010 20:24:09 -0400 (EDT)
Received: from hipster.bestpractical.com ([127.0.0.1]) by localhost (hipster.bestpractical.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id dWukXMCUO5K4 for <cpan-bug+Perl-Critic [...] hipster.bestpractical.com>; Mon, 5 Jul 2010 20:24:07 -0400 (EDT)
Received: from la.mx.develooper.com (x1.develooper.com [207.171.7.70]) by hipster.bestpractical.com (Postfix) with SMTP id A242661E007 for <bug-Perl-Critic [...] rt.cpan.org>; Mon, 5 Jul 2010 20:24:05 -0400 (EDT)
Received: (qmail 8169 invoked by uid 103); 6 Jul 2010 00:25:33 -0000
Received: from x16.dev (10.0.100.26) by x1.dev with QMQP; 6 Jul 2010 00:25:33 -0000
Received: from mailout2-6.pacific.net.au (HELO mailout2.pacific.net.au) (61.8.2.229) by 16.mx.develooper.com (qpsmtpd/0.80) with ESMTP; Mon, 05 Jul 2010 17:25:32 -0700
Received: from mailproxy1.pacific.net.au (mailproxy1.pacific.net.au [61.8.2.162]) by mailout2.pacific.net.au (Postfix) with ESMTP id BE80621820B for <bug-Perl-Critic [...] rt.cpan.org>; Tue, 6 Jul 2010 10:25:27 +1000 (EST)
Received: from blah.blah (ppp2402.dyn.pacific.net.au [61.8.36.2]) by mailproxy1.pacific.net.au (Postfix) with ESMTP id CE44C8C2E for <bug-Perl-Critic [...] rt.cpan.org>; Tue, 6 Jul 2010 10:25:22 +1000 (EST)
Received: from gg by blah.blah with local (Exim 4.72) (envelope-from <gg [...] zip.com.au>) id 1OVvxR-0000uK-4m for bug-Perl-Critic [...] rt.cpan.org; Tue, 06 Jul 2010 10:24:17 +1000
Delivered-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
User-Agent: Gnus/5.110011 (No Gnus v0.11) Emacs/23.2 (gnu/linux)
Subject: ProhibitLvalueSubstr docs
Return-Path: <gg [...] zip.com.au>
X-RT-Mail-Extension: perl-critic
X-Original-To: cpan-bug+Perl-Critic [...] hipster.bestpractical.com
X-Spam-Check-BY: 16.mx.develooper.com
Date: Tue, 06 Jul 2010 10:24:17 +1000
X-Spam-Level:
To: bug-Perl-Critic [...] rt.cpan.org
From: Kevin Ryde <user42 [...] zip.com.au>
X-RT-Original-Encoding: us-ascii
Content-Length: 605
Download (untitled) / with headers
text/plain 605b
The pod of ProhibitLvalueSubstr could note that the 4-arg substr is new in perl 5.005, so you know you might be upping your minimum by switching to it. But as an encouragement to go 4-arg, which is the purpose of the policy :-), it could be worth noting there's a misfeature in the lvalue form when used on magical or overloaded scalars -- they end up kept alive in the function scratchpad when you otherwise thought you'd dropped all references to them. Per my ticket http://rt.perl.org/rt3/Public/Bug/Display.html?id=67838 which I think has been acknowledged as real, but perhaps not fixed yet.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.