diff options
| author | Patrick Pannuto <ppannuto@codeaurora.org> | 2010-08-04 14:30:59 -0700 |
|---|---|---|
| committer | David Keitel <dkeitel@codeaurora.org> | 2016-03-23 20:51:40 -0700 |
| commit | 84ea4512d986bd6ee5482525880602f76f5d54d3 (patch) | |
| tree | 73836a15fbb2d221b031ba3b00545e67a2be4077 /scripts | |
| parent | 769b0796d59658ecac2be4313eda52874acedd67 (diff) | |
Checkpatch: Print location when catching return code error
Display context so users can see where there error was
OLD:
ERROR: illegal return value, please use an error code
NEW:
ERROR: illegal return value, please use an error code
#152: test.c:5:
+ return -1;
Change-Id: I098004d9a5dbeb6c39b35b84ac94fd7a861849d7
Signed-off-by: Patrick Pannuto <ppannuto@codeaurora.org>
(cherry picked from commit 8d09803de5d629f3ebc9cb357a40b5937a5d171e)
Signed-off-by: Stepan Moskovchenko <stepanm@codeaurora.org>
Diffstat (limited to 'scripts')
| -rwxr-xr-x | scripts/checkpatch.pl | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/scripts/checkpatch.pl b/scripts/checkpatch.pl index 0939b70a4230..7057ee01178b 100755 --- a/scripts/checkpatch.pl +++ b/scripts/checkpatch.pl @@ -5590,7 +5590,7 @@ sub process { # check for return codes on error paths if ($line =~ /\breturn\s+-\d+/) { ERROR("NO_ERROR_CODE", - "illegal return value, please use an error code"); + "illegal return value, please use an error code\n" . $herecurr); } # check for gcc specific __FUNCTION__ |