szepet added inline comments.

> alexfh wrote in SuspiciousEnumUsageCheck.cpp:155
> Why?

Because the hasDisjointValueRange function could not decide the values 
properly. So in case of an empty Enum it would not make sense. Fortunately we 
know that the empty case should not be reported so used early return on this.

That is why this is needed if we want a deterministic check.

https://reviews.llvm.org/D22507



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to