Bug 3176 - Invisible text that should be ignored is not.
Summary: Invisible text that should be ignored is not.
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: spamassassin (show other bugs)
Version: SVN Trunk (Latest Devel Version)
Hardware: Other other
: P5 normal
Target Milestone: 3.0.0
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-03-15 06:50 UTC by David Koppelman
Modified: 2004-03-17 09:59 UTC (History)
0 users


Attachment Type Modified Status Actions Submitter/CLA Status
Proposed fix. patch None David Koppelman [HasCLA]
Proposed fix. patch None David Koppelman [HasCLA]
Show Obsolete (1) Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description David Koppelman 2004-03-15 06:50:19 UTC 
With a recent change (r9460, Bug 2129) text invisible to the user
should be ignored by the Bayesian classifier.  However it appears that
the Bayesian classifier is seeing this text due to a bug in
Message::get_visible_rendered_body_text_array.  See the patch below.

I'm sure about the patch, but I'm not sure why this hasn't been seen,
it may be something about other non-trunk changes in my copy.
Comment 1 David Koppelman 2004-03-15 06:50:54 UTC 
Created attachment 1843 [details]
Proposed fix.
Comment 2 David Koppelman 2004-03-15 06:58:58 UTC 
Created attachment 1844 [details]
Proposed fix.

The previous patch only fixes the problem with learning, this patch
fixes learning and checking.
Comment 3 Justin Mason 2004-03-17 18:59:21 UTC 
ok, fixed in current SVN separately