walkero
/
cppcheck
1
0
Fork 0
Code Issues Pull Requests Projects Releases 1 Wiki Activity

htmlreport: only include verbose messages if they differ from the regular/actual message.

This commit is contained in:
Matthias Krüger 2014-11-08 23:48:32 +01:00
parent ffe7f57f4c
commit e730f525ae
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
htmlreport/cppcheck-htmlreport
View File

@ -268,13 +268,13 @@ class AnnotateCodeFormatter(HtmlFormatter):
if error['line'] == line_no: if error['line'] == line_no:
try: try:
if error['inconclusive'] == 'true': if error['inconclusive'] == 'true':
if error.get('verbose'): if error.get('verbose') and (error['verbose'] != error['msg']): # only print verbose msg if it really differs from actual message
index = t.rfind('\n') index = t.rfind('\n')
t = t[:index] + HTML_EXPANDABLE_INCONCLUSIVE % (error['msg'], html_escape(error['verbose'].replace("\\012", '\n'))) + t[index+1:] t = t[:index] + HTML_EXPANDABLE_INCONCLUSIVE % (error['msg'], html_escape(error['verbose'].replace("\\012", '\n'))) + t[index+1:]
else: else:
t = t.replace('\n', HTML_INCONCLUSIVE % error['msg']) t = t.replace('\n', HTML_INCONCLUSIVE % error['msg'])
except KeyError: except KeyError:
if error.get('verbose'): if error.get('verbose') and (error['verbose'] != error['msg']):
index = t.rfind('\n') index = t.rfind('\n')
t = t[:index] + HTML_EXPANDABLE_ERROR % (error['msg'], html_escape(error['verbose'].replace("\\012", '\n'))) + t[index+1:] t = t[:index] + HTML_EXPANDABLE_ERROR % (error['msg'], html_escape(error['verbose'].replace("\\012", '\n'))) + t[index+1:]
else: else: