On 2026-05-12 15:04:37 +0200, Vincent Lefevre wrote:
> On 2026-05-12 14:52:13 +0200, Vincent Lefevre wrote:
> > On 2026-05-12 14:24:47 +0200, Vincent Lefevre wrote:
> > > I can also reproduce on Debian/unstable with the
> > > spamassassin 4.0.2+svn1933012-1 Debian package.
> > 
> > If I output $value in html_range, I get 16 for the max_size test.
> > This is probably due to the fact that HTML.pm contains:
> > 
> >   $self->{max_size} = 16;       # start at default size
> > 
> > and no font sizes appear in the HTML file.
> 
> In any case, 16 will always be too large. On Debian 12 (bookworm),
> with spamassassin 4.0.1-1~deb12u1, there was
> 
>   $self->{max_size} = 3;        # start at default size
>   $self->{min_size} = 3;        # start at default size
> 
> while one now has
> 
>   $self->{max_size} = 16;       # start at default size
>   $self->{min_size} = 16;       # start at default size
> 
> Since 3 is the default (for <font size=...>), why has this been
> changed to 16?

The concerned commit is revision 1924977. Old sizes in
file lib/Mail/SpamAssassin/HTML.pm have been changed to
"font size in pixels" (whatever this means). But the rules
have not been updated: rules/20_html_tests.cf still has

body HTML_FONT_SIZE_LARGE  eval:html_range('max_size', '5', '6')
body HTML_FONT_SIZE_HUGE   eval:html_range('max_size', '6', 'inf')

i.e. refering to the old sizes as in <font size=...>.

-- 
Vincent Lefèvre <[email protected]> - Web: <https://www.vinc17.net/>
100% accessible validated (X)HTML - Blog: <https://www.vinc17.net/blog/>
Work: CR INRIA - computer arithmetic / Pascaline project (LIP, ENS-Lyon)

Reply via email to