develooper Front page | perl.perl5.porters | Postings from February 2009

Re: deprecating Switch.pm

Thread Previous | Thread Next
From:
Nicholas Clark
Date:
February 23, 2009 02:56
Subject:
Re: deprecating Switch.pm
Message ID:
20090223105645.GA81285@plum.flirble.org
On Mon, Feb 23, 2009 at 10:46:29AM +0000, Nicholas Clark wrote:
> I did this:
> 
> commit e76b2c0c0a5a6bcf7f940f55f050b6f3cecac437
> Author: Nicholas Clark <nick@ccl4.org>
> Date:   Mon Feb 23 11:27:08 2009 +0100
> 
>     Add deprecate.pm. Deprecate shipping Switch.pm in the core distribution.
> 
> 
> Switch changes like this:
> 
> diff --git a/lib/Switch.pm b/lib/Switch.pm
> index 709442e..8e98d29 100644
> --- a/lib/Switch.pm
> +++ b/lib/Switch.pm
> @@ -4,8 +4,10 @@ use strict;
>  use vars qw($VERSION);
>  use Carp;
>  
> -$VERSION = '2.14';
> +use if $] >= 5.011, 'deprecate';
>  
> +$VERSION = '2.14_01';
> +  

> Anyway, help wanted. Does anyone want to do any of:
> 
> 1: Write tests
> 2: Write documentation
> 3: Fix the bit of the code that is commented as "fragile"
> 4: Write a better warning message
> 5: Decide whether the core is allowed to issue multi-line diagnostic messages
> 6: Propose a patch for CPAN.pm
> 7: Propose a patch for CPANPLUS.pm
> 8: Comment further on the @INC order thread

Oh yes.

9: Is it possible to have a warning that defaults to on, so that one has to
   explicitly go no warnings 'deprecated' to shut it up?

because right now the behaviour is:

$ /Users/nick/Sandpit/511v/bin/perl5.11.0 -e 'use warnings; use Switch'
Switch will be removed from the Perl core distribution in the next major release. Please install it from CPAN. It is being used at -e line 1
$ /Users/nick/Sandpit/511v/bin/perl5.11.0 -e 'use Switch; use warnings'
$ 

so if they don't turn on warnings, or simple have it last in a list of modules
to use, they won't get the message.

Nicholas Clark

Thread Previous | 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