i would like point out one other way in which trivial patches are
problematic: they make it hard to do patches across branches. i have
to manually merge a patch because we checked in change into trunk that
added accessor functions. accessor functions are great, and i wouldn't
be so annoyed if it was part of a larger patch that motivated the
need, but it wasn't. it is of course just my opinion, but it is
something to think about before we commit a patch.