Attachment 27518 Details for Bug 12131 – Bug 12131: Remove unused dependency on Exporter

[patch] Bug 12131: Remove unused dependency on Exporter

Bug-12131-Remove-unused-dependency-on-Exporter.patch (text/plain), 1.45 KB, created by Marcel de Rooy on 2014-04-24 06:47:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Marcel de Rooy

Created: 2014-04-24 06:47:14 UTC

Size: 1.45 KB

patch

obsolete

>From 3e2b88b2e476e4c398e1687aa2257090fead340d Mon Sep 17 00:00:00 2001
>From: Colin Campbell <colin.campbell@ptfs-europe.com>
>Date: Wed, 23 Apr 2014 14:23:28 +0100
>Subject: [PATCH] Bug 12131: Remove unused dependency on Exporter
>Content-Type: text/plain; charset=utf-8
>
>C4::TmplToken depends on Exporter but does not use any
>of its facilities. Removed the dead code so it does not
>clog up the source and waste compilation time
>
>Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
>Module still compiles correctly; t/TmplToken.t passes.
>---
> C4/TmplToken.pm |   15 ++-------------
> 1 files changed, 2 insertions(+), 13 deletions(-)
>
>diff --git a/C4/TmplToken.pm b/C4/TmplToken.pm
>index 819d304..2283d16 100644
>--- a/C4/TmplToken.pm
>+++ b/C4/TmplToken.pm
>@@ -19,13 +19,8 @@ package C4::TmplToken;
> 
> 
> use strict;
>-#use warnings; FIXME - Bug 2505
>+use warnings;
> use C4::TmplTokenType;
>-require Exporter;
>-
>-use vars qw($VERSION @ISA @EXPORT @EXPORT_OK %EXPORT_TAGS);
>-
>-###############################################################################
> 
> =head1 NAME
> 
>@@ -37,14 +32,8 @@ This is a class representing a token scanned from an HTML::Template .tmpl file.
> 
> =cut
> 
>-###############################################################################
>+our $VERSION = 3.07.00.049;
> 
>-$VERSION = 3.07.00.049;
>-
>-@ISA = qw(Exporter);
>-@EXPORT_OK = qw();
>-
>-###############################################################################
> 
> sub new {
>     my $this = shift;
>-- 
>1.7.7.6

Actions: View | Diff | Splinter Review

Attachments on bug 12131: 27477 | 27518 | 27608