[PATCH] D23087: [LLVM-COV] Replace tabs to 2-space indentation in the HTML coverage report.

Ying Yi via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 2 15:26:03 PDT 2016


MaggieYi created this revision.
MaggieYi added reviewers: vsk, davidxl, bogner, harlanhaskins.
MaggieYi added a subscriber: llvm-commits.

When using llvm-cov.exe to generate the HTML report, the HTML report can look quite different to the source file if it includes tabs. The patch includes:
1) Use the default tab size is 2 spaces instead of 8 spaces. 
2) Add a command line switch (-tab-size) that allows user to set the tab size.

https://reviews.llvm.org/D23087

Files:
  test/tools/llvm-cov/Inputs/showTabsHTML.covmapping
  test/tools/llvm-cov/Inputs/showTabsHTML.proftext
  test/tools/llvm-cov/showTabsHTML.cpp
  tools/llvm-cov/CodeCoverage.cpp
  tools/llvm-cov/CoverageViewOptions.h
  tools/llvm-cov/SourceCoverageViewHTML.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D23087.66566.patch
Type: text/x-patch
Size: 6076 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160802/f23d0a0a/attachment.bin>


More information about the llvm-commits mailing list