Skip Menu |
 
rt.cpan.org will be shut down on March 1st, 2021.

This queue is for tickets about the Net-CIDR-Compare CPAN distribution.

Report information
The Basics
Id: 32069
Status: open
Priority: 0/
Queue: Net-CIDR-Compare

People
Owner: Nobody in particular
Requestors: cub.uanic [...] gmail.com
Cc:
AdminCc:

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



Subject: DESTROY is not a valid Net::CIDR::Compare macro
Download (untitled) / with headers
text/plain 451b
Great module, thanks! Unfortunately, just before finishing my program get following error: (in cleanup) DESTROY is not a valid Net::CIDR::Compare macro at ./aggregate_nets line 0 Versions that I use: Gentoo Linux 2.6.23-gentoo-r3, profiles/default-linux/x86/2007.0 perl v5.8.8 Net::CIDR v0.11 Net::CIDR::Compare v0.03 Net::Netmask v1.9015 My program is simple networks aggregation (see in attached file).
Subject: aggregate_nets
Download aggregate_nets
application/octet-stream 403b

Message body not shown because it is not plain text.

Subject: tst
Download tst
application/octet-stream 201b

Message body not shown because it is not plain text.

Download (untitled) / with headers
text/plain 225b
Moreover.... cub@tux ~ % cat q #!/usr/bin/perl -w use Net::CIDR::Compare; my $nets = Net::CIDR::Compare->new(); cub@tux ~ % perl q (in cleanup) DESTROY is not a valid Net::CIDR::Compare macro at q line 0 cub@tux ~ %
Download (untitled) / with headers
text/plain 387b
Adding empty destructor solves problem, and all works as expected. cub@tux ~ % diff -u Compare.pm.orig Compare.pm --- Compare.pm.orig 2008-01-03 23:13:25.000000000 +0200 +++ Compare.pm 2008-01-03 23:12:47.000000000 +0200 @@ -305,6 +305,8 @@ return \@result; } +sub DESTROY { +} # Autoload methods go after =cut, and are processed by the autosplit program. cub@tux ~ %
Subject: Re: [rt.cpan.org #32069] DESTROY is not a valid Net::CIDR::Compare macro
Date: Thu, 3 Jan 2008 16:41:35 -0500
To: bug-Net-CIDR-Compare [...] rt.cpan.org
From: "Gregory Jones" <grjones [...] gmail.com>
Download (untitled) / with headers
text/plain 650b
I'll add it to the next version. Thanks. On Jan 3, 2008 4:16 PM, Kostyuk Oleg via RT < bug-Net-CIDR-Compare@rt.cpan.org> wrote: Show quoted text
> > Queue: Net-CIDR-Compare > Ticket <URL: http://rt.cpan.org/Ticket/Display.html?id=32069 > > > Adding empty destructor solves problem, and all works as expected. > > cub@tux ~ % diff -u Compare.pm.orig Compare.pm > --- Compare.pm.orig 2008-01-03 23:13:25.000000000 +0200 > +++ Compare.pm 2008-01-03 23:12:47.000000000 +0200 > @@ -305,6 +305,8 @@ > return \@result; > } > > +sub DESTROY { > +} > > # Autoload methods go after =cut, and are processed by the autosplit > program. > > cub@tux ~ % >
Download (untitled) / with headers
text/plain 342b
I want to note that adding empty destructor - it's just dirty hack and not full decision. "DESTROY" should not be considered as "macro" in any way. I have no time to check all internals of your code, but I completely sure that code has some logical error. And I hope that you'll create proper patch instead of my hack. Thanks for your effort.


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.