* Daniel Walker <[email protected]> wrote:
> (btw, your comment above comes off very snide, which I don't
> appreciate. I haven't done anything that remotely warrants that.)
That snideness is simply a reflection, to make you realize how it feels.
I tried to point this issue out to you in the past, but here we go
again, back to square one. But i'm not giving up on you :-)
Peter sent a patch to a subtly buggy patch of yours (which introduces a
bug worse than it fixes), with this comment:
| | How about this - compile tested only
and you replied in this almost flagrant tone:
| How about a comment to go with it ? Which says something like what's
| above, notes on how the locking is getting used.. "
that kind of tone can be offensive, in such a context. I _know_ you
(still) dont realize it as offensive and that you dont accept my
characterisation of it, but nevertheless it's a fact and i'm going to
complain about it to you when i see you do it.
A proper polite answer to the helpful patch of Peter would have been
what i suggested:
> > how about: "if you've got some time then please also add a few
> > comments, because the code was quite non-obvious to me and I
> > misunderstood it when I tried to fix it. Thanks."
the basis of writing such replies is a certain level of humility and
fundamental respect towards the capabilities of other kernel developers.
Do you have it? If yes, are you willing to express it? If yes then
please do so.
Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
