Daniel Näslund wrote:
> Stefan Sperling wrote:
[...]
> > * subversion/tests/cmdline/patch_tests.py
> > (patch_with_fuzz): For determining fuzz we have an algorithm that
> > should only check for trailing and leading context. Make this test
> > verify that this property of the algorithm holds by adding context
> > in the middle of one of the hunks. Also fix an indentation error.
> >
> > What do you think?
>
> Looks good.
It's *beautiful*.
:-)
> This sounds silly but I often feel a bit of rush when I'm
> finished with a patch. It's like if I was thinking to myself: 'If I don't
> get it in immediately, the bug may disappear by itself'. I'm trying
> to slow down and give more thought to things. The most valueable lesson
> so far from contributing to subversion is the importance of beeing
> concious of all the choises I make when programming. In the end, writing
> a log message should be pretty easy if I really understand what I've
> been doing. :-)
I recognize those thoughts.
- Julian
Received on 2010-02-06 01:29:39 CET