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

Report information
The Basics
Id:
32069
Status:
open
Priority:
Low/Low

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

BugTracker
Severity:
(no value)
Broken in:
(no value)
Fixed in:
(no value)



Subject: DESTROY is not a valid Net::CIDR::Compare macro
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

Message body not shown because it is not plain text.

Subject: tst

Message body not shown because it is not plain text.

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 ~ %
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>
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 ~ %

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 runs on Request Tracker, is sponsored by The Perl Foundation, and maintained by Best Practical Solutions.

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