Skip to content

Commit b1a5332

Browse files
authored
Improved error message, fix typos (#387)
* Add location to error message. * Update default version ID * fix spelling Signed-off-by: Henry Cox <[email protected]>
1 parent 12dc677 commit b1a5332

File tree

5 files changed

+8
-7
lines changed

5 files changed

+8
-7
lines changed

Diff for: bin/get_version.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ else
2525
fi
2626

2727
# Fallback
28-
[ -z "$VERSION" ] && VERSION="2.3"
28+
[ -z "$VERSION" ] && VERSION="2.4"
2929
[ -z "$RELEASE" ] && RELEASE="beta"
3030
[ -z "$FULL" ] && FULL="$VERSION-$RELEASE"
3131

Diff for: lib/lcovutil.pm

+4-3
Original file line numberDiff line numberDiff line change
@@ -4676,9 +4676,10 @@ sub define_function
46764676
$location = '"' . $self->filename() . '":' . $start_line
46774677
unless defined($location);
46784678
lcovutil::ignorable_error($lcovutil::ERROR_INCONSISTENT_DATA,
4679-
"$location: duplicate function '$fnName' starts on line $start_line but previous definition started on "
4680-
. $data->line()
4681-
. MessageContext::context() . '.')
4679+
"$location: duplicate function '$fnName' starts on line \"" .
4680+
$data->filename() .
4681+
"\":$start_line but previous definition started on " .
4682+
$data->line() . MessageContext::context() . '.')
46824683
unless
46834684
grep({ $fnName =~ $_ } @lcovutil::suppress_function_patterns);
46844685
# if ignored, just return the function we already have -

Diff for: man/genhtml.1

+1-1
Original file line numberDiff line numberDiff line change
@@ -1022,7 +1022,7 @@ table entry to the first line in the corresponding file in the particular
10221022
category - e.g., to the first 'MIS' line (vanilla coverage report - see the
10231023
.i \-\-show\-navigation
10241024
option, below), to
1025-
the first 'UNC' branch (differential coverage repot), etc.
1025+
the first 'UNC' branch (differential coverage report), etc.
10261026
Hyperlinks from the summary table at the top of the 'source detail' page are not affected.
10271027

10281028
.RE

Diff for: rpm/lcov.spec

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
Summary: A graphical code coverage front-end
22
Name: lcov
3-
Version: 2.3
3+
Version: 2.4
44
Release: 1
55
License: GPLv2+
66
Group: Development/Tools

Diff for: tests/lcov/extract/extract.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -429,7 +429,7 @@ if [ 0 != $? ] ; then
429429
fi
430430
431431
432-
# workaround: depending on compiler verision, we see a coverpoint on the
432+
# workaround: depending on compiler version, we see a coverpoint on the
433433
# close brace line (gcc/6 for example) or we don't (gcc/10 for example)
434434
BRACE_LINE='^DA:34'
435435
MARKER_LINES=`grep -v $BRACE_LINE internal.info | grep -c "^DA:"`

0 commit comments

Comments
 (0)