Skip to main content
Active reading. [<http://stackoverflow.com/legal/trademark-guidance> (the last section)].
Source Link
Peter Mortensen
  • 31.6k
  • 4
  • 22
  • 14

Here are some resources about editing code in other peoplespeoples' questions. The first is the post which was linked in the comments on your question, and which I linked to previously. The other two were migrated to Meta StackExchangeStack Exchange when that site was split from Stack Overflow, so are a little harder to find if you just search Meta SO.

Here are some resources about editing code in other peoples questions. The first is the post which was linked in the comments on your question, and which I linked to previously. The other two were migrated to Meta StackExchange when that site was split from Stack Overflow, so are a little harder to find if you just search Meta SO.

Here are some resources about editing code in other peoples' questions. The first is the post which was linked in the comments on your question, and which I linked to previously. The other two were migrated to Meta Stack Exchange when that site was split from Stack Overflow, so are a little harder to find if you just search Meta SO.

Source Link

First off, thank you for asking. One of the best ways to get help if you're still unclear after reading the FAQs and other documentation is to ask here on Meta. (Ignore the downvotes.)


First edit: https://stackoverflow.com/review/suggested-edits/24698499

  • Good:

  • Fixed excessive indentation of code block

  • Improper capitalization of 'Underline' and 'Italics' fixed

  • Bad:

  • "Till" was grammatically correct; replaced with "til" which is informal/colloquial/many consider incorrect

  • Appended :; at the end of a sentence; this was probably intended to just be : but was typo'd

No other potential issues with the text.

This is the most "borderline" of these suggested edits that I can see. Overall the code indentation fix did make the post significantly more readable, but the replacement of the valid English word "till" with the less-appropriate "til" was not. (I'm considering the :; appended to the sentence as an honest typo instead of an intentional change.)

I personally would have selected 'reject and edit' to roll back the till/til change and fix the typo.

Second edit: https://stackoverflow.com/review/suggested-edits/24765018

As mentioned in the comments, changing the code style of someone's post is a big no-no, especially if the change is simply to your preference. Recommended reading is this post: When is it okay to edit posts for "Code Formatting?"

  • Good:
  • Fixed some slightly awkward phrasing for clarity ("This will just make the default constructor not being called." became: "This will just prevent the default constructor from being called.")
  • Bad:
  • Changed the original code style
  • Unclear edit summary: "Formatting, backticks, etc".

As mentioned, changing the code style is pretty egregious; I would have rejected due to that as well because it didn't make the post any more readable than previously. The edit summary also doesn't reflect your changes; keeping in mind the character limit, I would have said something like "Improved code formatting and reworded sentence to make it clearer."

(The grammatical fix for the mentioned sentence did make the point much clearer, in my opinion, though; I think it's important to point out that this sort of edit is quite helpful.)

Some people will argue that the highlighting of the class names throughout the answer using code formatting makes it less readable. I personally prefer that the class names be highlighted consistently throughout the answer so I could track them easily; not everyone agrees with this point of course.

Third edit: https://stackoverflow.com/review/suggested-edits/24764959

  • Good:
  • Fixed improper code formatting using <pre> and <code> HTML sections.
  • Fixed all of the improper grammar present in the post: improper capitalization, missing punctuation, etc.
  • Bad:
  • Changed the original code style

If you had left the code as written instead of changing the positions of the opening braces, and just replaced the improper HTML tags with proper code blocks (three backticks or four spaces), I would have accepted this edit. However changing the code format for no reason as far as I can tell didn't help the answer's readability at all, and should be avoided.


As an addendum, people are extremely sensitive to editors changing the code of other peoples' answers or questions. Basic things like fixing excessive whitespace (eg the whole block is indented 16 spaces for some reason) is fine; but when you actually start changing the contents of the code block itself is when you need to be extra careful and have a very good reason to do so. Some reviewers might have found updating the brace placement to match the C# code standards to be acceptable; your reviewers (and others, myself included) do not.

There are numerous questions here on Meta regarding edits to code, whether they were appropriate, what to do in various situations, etc.

Here are some resources about editing code in other peoples questions. The first is the post which was linked in the comments on your question, and which I linked to previously. The other two were migrated to Meta StackExchange when that site was split from Stack Overflow, so are a little harder to find if you just search Meta SO.