[EMAIL PROTECTED] writes:

> Author: felicity
> Date: Thu Jun  3 12:19:51 2004
> New Revision: 20793
> 
> Modified:
>    incubator/spamassassin/trunk/t/bayessql.t
> Log:
> bug 3465: typo in bayessql test got my() repeated warnings

This is simple, but it technically needed prior review, so I'll use this
bug as a good point to post about this -- been discussing it a bit on
IRC/AIM and also talked with Ken Coar (the incubator chair) a bit on
#apache.  Basically, I think it would help if we articulate which
changes need R-T-C and which don't.

R-T-C optional:
  - documentation
  - finishing off pre-existing T_ tests
  - non-controversial non-semantic style changes (fixing indentation,
    adding comments, but not actual code)
  - very simple, non-controversial, and absolutely safe bug fixes (I
    think this check-in is a good example, but any more would need a
    review)

R-T-C required:
  - everything else

-- 
Daniel Quinlan
http://www.pathname.com/~quinlan/

Reply via email to