Skip to main content
8 events
when toggle format what by license comment
May 12, 2014 at 21:44 comment added Robert Harvey Mod Eh, I changed my mind. I'm not entirely sure that the original edit worked anyway.
May 12, 2014 at 21:15 comment added Servy @RobertHarvey I would have made such an edit, if it weren't for the comment from the post author indicating that he feels that the edit isn't appropriate. I would have assumed, if he recognized the simple mistake as a mistake, he would have simply fixed it himself at that point.
May 12, 2014 at 20:16 vote accept Aurimas
May 12, 2014 at 20:16 comment added Aurimas Thanks @robert-harvey. I see that I could have done a less intrusive edit to convey the same idea and perhaps that would have passed review. I'll keep that in mind in the future.
May 12, 2014 at 20:06 comment added Robert Harvey Mod While suggested editors may see these kinds of suggestions declined, users with editing privileges can make these changes unilaterally, barring some kind of rollback. I went ahead and made a similar edit myself, and notified the poster.
May 12, 2014 at 20:02 comment added Servy @RobertHarvey I can certainly understand rejecting this edit; I agree it's not a clear cut case. To me personally, while the number of characters of code edited isn't tiny, the intention of the programmer seems to clearly be that the regex is executed on each iteration, so fixing that isn't just making the code work, it's making the code do what the author meant it to do. I don't see it as the authors intention to execute the regex once and then iterate on that one result over and over. I was, of course, wrong, and it seems that is his intention, although I have no idea why.
May 12, 2014 at 19:59 comment added Robert Harvey Mod I also agree that not all code edits are bad. This particular one was probably substantial enough to justify its own answer with detailed explanations (it's not merely a case of a misspelled identifier, or a misplaced comma or semicolon; it actually changes the code logic).
May 12, 2014 at 19:57 history answered Servy CC BY-SA 3.0