[development] Trust (was: Hit and run contrib)

Eric-Alexander Schaefer eric.schaefer at eas-consulting.de
Sat May 31 15:44:17 UTC 2008


Jakob Petsovits schrieb:
> On the other hand, just recently I noticed a missing break; statement 
> somewhere in Views. Checked out with CVS. But it didn't directly concern me 
> or pose any problems to what I was doing, and opening up a new issue would 
> have taken more time and effort than I was willing to put in. Like, "someone 
> will probably fix it anyways". With commit access, I would have committed the 
> fix right away, and in the unlikely case that it's actually wrong (and a
> "// fall through" comment would be required instead) then Earl would have 
> noticed and fixed it the right way. It's probably still unfixed, I guess.

Would you fix the bug without opening an issue which documents it? That 
might be OK if you are on your own. But its a really bad idea in a team 
environment.
Views is a large module and if you do not grok it, how do you know the 
break isn't missing on purpose?

Eric


More information about the development mailing list