27

I recently suggested an edit that removes the bold formatting that was applied to all the text of an answer and fixes some minor typos and language errors. It was approved by one user and rejected by two. I made sure to re-read my edit to see if I introduced some language or formatting errors, but couldn't find any. What should I do in this case?

26
  • 31
    Imho: Two quite new reviewers made a wrong choice
    – BDL
    Sep 14 at 18:57
  • 10
    It does leave rather large grammatical errors uncorrected. I might have accepted and Improved Edit but I can see it reasonable to also reject.
    – VLAZ
    Sep 14 at 18:58
  • 6
    Oh wow. That entire Q&A has... issues. Sep 14 at 19:04
  • 2
    Not quite related to editing that post but...it doesn't make any sense. That's not a language issue, just the code doesn't even do what the description claims. Nor does the description tie in with the code. It talks about overriding a constructor but nothing like this happens. And the code seems to compile fine, not show an error as the text claims. Boggles the mind how that has three upvotes.
    – VLAZ
    Sep 14 at 19:07
  • 4
    One possibility is reviewers only looked at the rendered view, which does not accurately represent the diff (it only shows the holding change, nothing else).
    – nobody
    Sep 14 at 19:19
  • 1
    @MehdiCharife I mean only the first sentence is right. All the rest is wrong. Trying to match the constructor name of the parent and that being a method is a red herring. It's not at all the reason why constructor overriding is not possible. You can have the super and sub class with the same name - if they reside in different packages: class Foo extends com.other.Foo {} That won't make the constructor overriding possible. Methods and constructors are simply not trivially interchangeable.
    – VLAZ
    Sep 14 at 19:50
  • 2
    I don't know. I would have approved it. Sep 14 at 21:09
  • 1
    @nobody That's a really good callout; it's pretty unfortunate that removing the bold highlights the entire block in the diff and hides the specific changes within; at first glance it looks like basically nothing changed other than the bold... but I'd also argue that reviewers should be paying better attention than that, too.
    – zcoop98
    Sep 14 at 22:13
  • 1
    I made a further attempt to improve the grammar in the answer, because the grammar was really quite poor (and there was some redundancy). However, the answer seems bad anyway: the example doesn't actually appear to show an attempt to override a constructor; and if the description says that the compiler will complain and require something, it should show the form that the constructor complains about, not a fixed form. Sep 14 at 22:34
  • 3
    Yes, I like your wording better and approved it. But I also cast a delete vote (and several more) - good lord there are a lot of awful answers there. Sep 14 at 22:52
  • 7
    @AlexeiLevenkov I don't think the post in question is in any way random. It asks a FAQ question, and it was the first that popped up after I googled whether it was possible to override a constructor. Most of the editing that I do relates to Q&As concerning questions that I myself look for answers to. Sep 15 at 0:30
  • 7
    If the edit suggestion queue had a majority of entries of this quality we would be in a much better place. 90% of them are much worse. Sep 15 at 0:38
  • 4
    @AlexeiLevenkov The fact that my edits cover posts from different technologies doesn't necessarily mean that my basis for selecting those posts was random. I've been reading on design patterns and object-oriented design lately, and these subjects tend to be illustrated in a number of different languages and technologies. I'm also working on an ML project in C in which OOP is emulated, which partly explains the C edits. Sep 15 at 1:08
  • 3
    @SecurityHound the answer was from 2015, definitely not GenAI. Just regular human… output. Sep 15 at 18:52
  • 2
    @SecurityHound screenshot of the deleted answer. People have been wrong before ChatGPT existed.
    – VLAZ
    Sep 16 at 13:19

0

You must log in to answer this question.