hans accepted this revision.
hans added a comment.
This revision is now accepted and ready to land.

Looks fine to me.



================
Comment at: clang/docs/SourceBasedCodeCoverage.rst:330
+last case ends). This gap region has a zero count: this causes "gap" areas in
+between case statements, which contain no executable code, to appear uncovered.
+
----------------
I thought the point is that it //doesn't// appear uncovered in coverage 
reports? I.e. it will have a zero count, but that's expected.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70571/new/

https://reviews.llvm.org/D70571



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

Reply via email to