The currently highest voted unanswered question is, well, unanswerable. As Martin said one year ago, "there is actually nothing here to review" and "the actual code is missing as are some other parts like the definition of
The question has been voted to be closed several times, and every time the review queue votes to leave it open. Now, all those votes are based on personal preference, I guess. Some might see the high number of votes, the clear code style and say "this question is perfectly fine".
However, this particular question doesn't contain the necessary parts to get fully answered. Whether the "compile-time check[..]" works depends on
enum_length. The original author has asked at least one question about the implementation of
SETUP_ENUM_LENGTH, but our rules dictate that the code must be included in the question itself, not in the context of the question. Also note that
DEF_ENUM was taken from this answer without proper attribution, but whether or not a three line snippet needs that isn't really part of this question.
At least from my point of view the question lacks critical context and therefore should get closed. But that's my opinion. So I'm wondering: why is this question still open?