diff options
author | Eliseo Martínez <eliseomarmol@gmail.com> | 2014-11-04 15:41:48 +0100 |
---|---|---|
committer | Eliseo Martínez <eliseomarmol@gmail.com> | 2014-11-06 09:51:44 +0100 |
commit | 22475b5ae80b5c08115677b9abc5c62258716c08 (patch) | |
tree | 724cf7e7a10a0ff7a83daba559d834c2817f2a4f /test/functional/shell/viml_system_spec.lua | |
parent | 336aab5eef68a493dcd1ddc2dfe6405b6ee7a0a9 (diff) | |
download | rneovim-22475b5ae80b5c08115677b9abc5c62258716c08.tar.gz rneovim-22475b5ae80b5c08115677b9abc5c62258716c08.tar.bz2 rneovim-22475b5ae80b5c08115677b9abc5c62258716c08.zip |
Fix warnings: regexp.c: br_regcomp(): Np dereference: MI.
Problem: Dereference of null pointer @ 1312.
http://neovim.org/doc/reports/clang/report-b1d09a.html#EndPath
Diagnostic: Multithreading issue.
Rationale : Suggested error path contains two succesive calls to
`regnext(scan)`, first of which returning nonnull, the
second one returning null. This can only occur if global
`reg_toolong` accesed in `regnext()` changes between the
calls.
Resolution: Use local variable to cache first `regnext(scan)` result.
Note that this change alters function semantics, as now
function only issues one call instead of two, reusing the
result for the second time.
This shouldn't be a problem, though, as new semantics should
be in fact be better.
Diffstat (limited to 'test/functional/shell/viml_system_spec.lua')
0 files changed, 0 insertions, 0 deletions