Skip to content

Commit e3e917f

Browse files
committed
adapt commentary and make notes more concise
1 parent 2c2d23e commit e3e917f

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

cmake/script/CoverageInclude.cmake.in

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@ separate_arguments(LCOV_OPTS)
2424
set(LCOV_COMMAND ${LCOV_EXECUTABLE} --gcov-tool ${CMAKE_CURRENT_LIST_DIR}/cov_tool_wrapper.sh ${LCOV_OPTS})
2525

2626
find_program(GENHTML_EXECUTABLE genhtml REQUIRED)
27+
28+
# HTML_OPTS is optionally passed via -D flag.
2729
# If HTML_OPTS is not provided, implicitly pass LCOV_OPTS to genhtml.
2830
separate_arguments(HTML_OPTS)
2931
set(GENHTML_OPTS ${HTML_OPTS})

doc/developer-notes.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -508,8 +508,7 @@ enabled by setting `LCOV_OPTS="--rc branch_coverage=1"`:
508508
cmake -DLCOV_OPTS="--rc branch_coverage=1" -P build/Coverage.cmake
509509
```
510510

511-
HTML_OPTS can be specified to provide an options override to genhtml from the default LCOV_OPTS, the program that generates the
512-
html report from lcov: `HTML_OPTS="--exclude boost"`.
511+
HTML_OPTS can override the genhtml options (which default to LCOV_OPTS).
513512

514513
```
515514
cmake -DHTML_OPTS="--exclude boost" -P build/Coverage.cmake

0 commit comments

Comments
 (0)