[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/