diff options
author | Eliseo Martínez <eliseomarmol@gmail.com> | 2014-11-14 20:44:20 +0100 |
---|---|---|
committer | Eliseo Martínez <eliseomarmol@gmail.com> | 2014-11-18 21:57:47 +0100 |
commit | 04e42f2ae409294d551c9b589aee4cbfd2616d68 (patch) | |
tree | b815cb86e7b165ce5e9685d1467d85f1a8f3c99f /src/nvim/api/vim.c | |
parent | 9e37c1d3b6b9d0d35d7f7558d3efb555757278c0 (diff) | |
download | rneovim-04e42f2ae409294d551c9b589aee4cbfd2616d68.tar.gz rneovim-04e42f2ae409294d551c9b589aee4cbfd2616d68.tar.bz2 rneovim-04e42f2ae409294d551c9b589aee4cbfd2616d68.zip |
Fix warnings: tag.c: test_for_static()/get_tags(): Various (2): FP.
Problems : Assigned value is garbage or undefined @ 2191.
Uninitialized argument value @ 2796.
Diagnostic : False positives.
Rationale : Both problems share the same cause.
Error happens in get_tags(), if parse_match() fails because
of parse_tag_line() failing before. Then, `tp` is not
correctly initialized and subsequent code accesses garbage
values.
This is not really possible, as parse_tag_line() should not
fail after find_tags() has been successful.
That is because find_tags() already does tag line parsing,
using parse_tag_line() itself for it (or a quicker
alternative that should produce same result). That's why
return value of parse_match() is ignored, and subsequent
code assumes it is successful.
Resolution : Assert parse_match() always successful.
Diffstat (limited to 'src/nvim/api/vim.c')
0 files changed, 0 insertions, 0 deletions