Use Module::Runtime instead of eval "require" or eval "use"

RESOLVED FIXED

Status

()

--
enhancement
RESOLVED FIXED
8 years ago
5 months ago

People

(Reporter: mkanat, Assigned: dylan)

Tracking

Production

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Reporter)

Description

8 years ago
There's a nice module called Class::Load on CPAN:

  http://search.cpan.org/dist/Class-Load/lib/Class/Load.pm

It does class loading, and has functions to check if classes are loaded. It should be faster and cleaner than using evals, and we're doing this in so many places now that we should have a better solution.
(Assignee)

Comment 1

5 months ago
I'm tired of the number of warnings I get from perl critic. Fixing this now.
Assignee: general → nobody
Component: Bugzilla-General → General
Product: Bugzilla → bugzilla.mozilla.org
QA Contact: default-qa
Summary: Use Class::Load instead of eval { require } or eval "use" → Use Module::Runtime instead of eval { require } or eval "use"
Version: 3.2 → Production
(Assignee)

Updated

5 months ago
Assignee: nobody → dylan
(Assignee)

Comment 2

5 months ago
Posted file GitHub Pull Request
(Assignee)

Updated

5 months ago
Status: NEW → RESOLVED
Last Resolved: 5 months ago
Resolution: --- → FIXED
(Assignee)

Updated

5 months ago
Summary: Use Module::Runtime instead of eval { require } or eval "use" → Use Module::Runtime instead of eval "require" or eval "use"
You need to log in before you can comment on or make changes to this bug.