[Rt-commit] rt branch, 4.2/html-gumbo-loading, created. rt-4.2.5-115-g1aefd60

Wallace Reis wreis at bestpractical.com
Wed Jul 2 14:40:43 EDT 2014


The branch, 4.2/html-gumbo-loading has been created
        at  1aefd600fcbb5ded14376f77d77a14758668fb39 (commit)

- Log -----------------------------------------------------------------
commit 1aefd600fcbb5ded14376f77d77a14758668fb39
Author: Wallace Reis <wreis at bestpractical.com>
Date:   Wed Jul 2 15:19:50 2014 -0300

    Use UNIVERSAL::require for loading HTML::Gumbo
    
    ... for consistency, like we already do in some other places instead of
    using an eval. This is a residual change from 4.2/html-gumbo-balancing.

diff --git a/lib/RT/Interface/Web.pm b/lib/RT/Interface/Web.pm
index 6b87aa8..f82cf57 100644
--- a/lib/RT/Interface/Web.pm
+++ b/lib/RT/Interface/Web.pm
@@ -3971,7 +3971,7 @@ sub _NewScrubber {
     require HTML::Scrubber;
     my $scrubber = HTML::Scrubber->new();
 
-    if (eval "require HTML::Gumbo; 1") {
+    if (HTML::Gumbo->require) {
         no warnings 'redefine';
         my $orig = \&HTML::Scrubber::scrub;
         *HTML::Scrubber::scrub = sub {

-----------------------------------------------------------------------


More information about the rt-commit mailing list