Permalink
Browse files

remove_pattern subroutine didn't work to exclude iOS SDK; inline it

  • Loading branch information...
1 parent 9e763ab commit c90d88372e71734abd7f65501ab74590c2293c30 @jonreid committed Nov 30, 2012
Showing with 4 additions and 9 deletions.
  1. +1 −1 README.md
  2. +3 −8 getcov
View
@@ -33,7 +33,7 @@ Modification
There are two places you may want to modify:
1. In envcov.sh, ``LCOV_INFO`` determines the name shown in the report.
-2. In getcov, edit ``exclude_data()`` to change what code to exclude, for
+2. In getcov, edit ``exclude_data()`` to specify which files to exclude, for
example, third-party libraries.
View
11 getcov
@@ -28,16 +28,11 @@ gather_coverage()
"${LCOV_PATH}/lcov" -b "${SRCROOT}" -d "${OBJ_DIR}" --capture -o ${LCOV_INFO}
}
-remove_pattern()
-{
- "${LCOV_PATH}/lcov" -d "${OBJ_DIR}" --remove ${LCOV_INFO} $@ -o ${LCOV_INFO}
-}
-
exclude_data()
{
- remove_pattern "/Applications/Xcode.app/*"
- remove_pattern "main.m"
- # Remove other patterns if you like...
+ "${LCOV_PATH}/lcov" -d "${OBJ_DIR}" --remove ${LCOV_INFO} "/Applications/Xcode.app/*" -o ${LCOV_INFO}
+ "${LCOV_PATH}/lcov" -d "${OBJ_DIR}" --remove ${LCOV_INFO} "main.m" -o ${LCOV_INFO}
+ # Remove other patterns here...
}
generate_report()

0 comments on commit c90d883

Please sign in to comment.