Re: svn commit: r372080 - /spamassassin/trunk/lib/Mail/SpamAssassin/HTML.pm

[Theo Van Dinter <fe...@apache.org>]

On Wed, Jan 25, 2006 at 02:48:59AM -0000, jm@apache.org wrote:
> +  # bug 4695: we want "<br/>" to be treated the same as "<br>", and
> +  # the HTML::Parser API won't do it for us
> +  # if ($self->{preclean_empty_elements}) {
> +  $text =~ s/<br\s*\/>/<br>/gis;
> +  # }

I think we need to be more generic, handling any /\s*\/>/ --
the same problem occurs with other tags such as "<p/>", etc.
A still-not-100%-but-closer version would be:

$text =~ s@<([^>]*)\s*/>@<$1>@g;

-- 
Randomly Generated Tagline:
""  "    ""   " "    "   "" " (random quotes)