http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4969

           Summary: M::SA::Message::Node::find_parts() has useless recursive
                    option
           Product: Spamassassin
           Version: 3.1.3
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: normal
          Priority: P5
         Component: Libraries
        AssignedTo: [email protected]
        ReportedBy: [EMAIL PROTECTED]


While working on implementing a rule suggestion that came up on the mailing
list, I was noticing that the recursive parameter as implemented by find_parts()
is rather useless -- if true (the default) all subchildren are searched (which
is what we usually want to do), however if false, no subchildren are searched
which means only the single part is looked at -- which we could easily do
without find_parts.

So I'm going to attach a patch shortly which changes the recursive == false
option to mean "look at the current part and any direct children, but do not
recurse further" which is really more of what I meant during the initial
implementation.



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Reply via email to