Closed Bug 623384 Opened 14 years ago Closed 6 years ago

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

Categories

(bugzilla.mozilla.org :: General, enhancement)

Production
enhancement
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: mkanat, Assigned: dylan)

Details

Attachments

(1 file)

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.
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: nobody → dylan
Attached file GitHub Pull Request
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
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.

Attachment

General

Created:
Updated:
Size: