5

This question already has an answer here:

I came across this answer in the review queue:

Sort the other range only, and then use std::merge.

I thought, that answer may be the solution, but instead of simply posting a link, the AP should elaborate a bit, adding even the smallest code example in case the linked resource ceases to exist.

When I did that, I saw the Big Bad Blame Screen of Death:

STOP! Look and Listen.

This was an audit, designed to see if you were paying attention. You didn't pass. Your review was inappropriate. This was a high quality post and you should have considered leaving it as-is or even upvoting.

OK, is adding a comment something so bad that I should be ashamed of trying to do that with so highly upvoted answer? Maybe it's time to fix review audits a bit?

--edit-- I've added my comment anyway!

marked as duplicate by psubsee2003, Lance Roberts, Martijn Pieters, hims056, Flyk Feb 12 '14 at 8:45

This question has been asked before and already has an answer. If those answers do not fully address your question, please ask a new question.

  • Related: meta.stackexchange.com/questions/191459/… – michaelb958 Feb 12 '14 at 8:01
  • @psubsee2003 however, this shows a specific case where a comment was appropriate. – John Dvorak Feb 12 '14 at 8:11
  • @JanDvorak it was a bad review, yes, but that's been discussed to death too. – psubsee2003 Feb 12 '14 at 8:12
  • @LowerClassOverflowian than what's the point of enabling comments/votes in review queue if it should not be used? The system expects me to take any action, or just hit 'It's OK'. In that case, it wasn't OK, otherwise I wouldn't be here. – Danubian Sailor Feb 12 '14 at 12:28