@raggi @s @b0rk intention (and how that intention is perceived) matters a lot, here.

I've def. been that jerk who made a patch just to get rid of the requester. Oft in times of stress.
That intention is bad.

But quite often "a patch says more than a thousand words", where the idea and lesson is best conveyed in an example. I then try to keep it an example, though. Not finished. "Look, this is the idea of dependency injection here. The actual implementation code needs work still"