Juan Lang wrote:
What this misses is the most common status that causes us all to argue:
uncomitted, because Alexandre's not sure about it. Perhaps he has a gut
feeling that the approach is not right, but hasn't taken the time to
identify any particular flaw. Perhaps it merits additional thought.
Perhaps it's too big to review quickly. Perhaps he hopes the submitter
will realize it's not up to snuff, and resubmit a better patch.
So what? just set "Rejected, reason: tests needed" or "will review later" or
whatever.
Ivan.