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 enum_length
".
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?
enum_length
is part of the interface viaSETUP_ENUM_LENGTH
. One could review the interface solely, but it's missing its documentation, so a review would most likely end in "that looks nice, but I have no idea what your code is supposed to do in scenario XY". It's not possible to reviewDEF_ENUM
, though. \$\endgroup\$