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