[pve-devel] applied: [PATCH v2 common] fix #1548: replace perl use vars with our
Thomas Lamprecht
t.lamprecht at proxmox.com
Thu Nov 15 09:57:45 CET 2018
On 11/15/18 9:47 AM, Tim Marx wrote:
> replace deprecated 'use vars qw($frob @mung %seen);' with the now in favor
> 'our $foo;' declaration
> see NOTE at: http://perldoc.perl.org/vars.html
>
> Signed-off-by: Tim Marx <t.marx at proxmox.com>
> ---
> changes from v1:
> * whitespace madness resolved
>
applied, thanks!
> src/PVE/AtomicFile.pm | 3 +--
> src/PVE/Exception.pm | 6 ++----
> src/PVE/SafeSyslog.pm | 10 +++-------
> 3 files changed, 6 insertions(+), 13 deletions(-)
>
> diff --git a/src/PVE/AtomicFile.pm b/src/PVE/AtomicFile.pm
> index b1889e8..2086c7a 100644
> --- a/src/PVE/AtomicFile.pm
> +++ b/src/PVE/AtomicFile.pm
> @@ -3,9 +3,8 @@ package PVE::AtomicFile;
> use strict;
> use warnings;
> use IO::AtomicFile;
> -use vars qw(@ISA);
>
> - at ISA = qw(IO::AtomicFile);
> +our @ISA = qw(IO::AtomicFile);
>
> sub new {
> my $class = shift;
> diff --git a/src/PVE/Exception.pm b/src/PVE/Exception.pm
> index fa6b73a..29fd94a 100644
> --- a/src/PVE/Exception.pm
> +++ b/src/PVE/Exception.pm
> @@ -6,12 +6,9 @@ package PVE::Exception;
>
> use strict;
> use warnings;
> -use vars qw(@ISA @EXPORT_OK);
> -require Exporter;
> use Storable qw(dclone);
> use HTTP::Status qw(:constants);
>
> - at ISA = qw(Exporter);
>
> use overload '""' => sub {local $@; shift->stringify};
> use overload 'cmp' => sub {
> @@ -20,7 +17,8 @@ use overload 'cmp' => sub {
> return "$a" cmp "$b"; # compare as string
> };
>
> - at EXPORT_OK = qw(raise raise_param_exc raise_perm_exc);
> +use base 'Exporter';
> +our @EXPORT_OK = qw(raise raise_param_exc raise_perm_exc);
>
> sub new {
> my ($class, $msg, %param) = @_;
> diff --git a/src/PVE/SafeSyslog.pm b/src/PVE/SafeSyslog.pm
> index 63b37f8..c61e97a 100644
> --- a/src/PVE/SafeSyslog.pm
> +++ b/src/PVE/SafeSyslog.pm
> @@ -5,16 +5,12 @@ use warnings;
> use File::Basename;
> use Sys::Syslog ();
> use Encode;
> +use base 'Exporter';
>
> -use vars qw($VERSION @ISA @EXPORT);
> +our $VERSION = '1.00';
>
> -$VERSION = '1.00';
>
> -require Exporter;
> -
> - at ISA = qw(Exporter);
> -
> - at EXPORT = qw(syslog initlog);
> +our @EXPORT = qw(syslog initlog);
>
> my $log_tag = "unknown";
>
>
More information about the pve-devel
mailing list