aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Fix warnings: screen.c: win_line(): Dead assigment: HI.Eliseo Martínez2014-11-06
| | | | | | | | | | | | Problem: Dead assigment. http://neovim.org/doc/reports/clang/report-7362ba.html#EndPath Diagnostic: Harmless issue. Rationale : `boguscols` is in fact unread by downstream code. Resolution: Comment out. This is preferred here over just removing the line because involved logic is complex, and future readers of this code could find this extra knowledge useful to understand what the code is doing.
* Fix warnings: screen.c: win_line(): Dead initialization: HI.Eliseo Martínez2014-11-06
| | | | | | | | | | | | Problem: Dead initialization @ 3477. http://neovim.org/doc/reports/clang/report-94b736.html#EndPath Diagnostic: Harmless issue. Rationale : `len` is assigned a new value just some lines below. So, this just seems something due to old-style variable declarations. Resolution: We could just remove initialization, but prefer moving declaration down to point of initialization.
* Fix warnings: screen.c: redraw_asap(): Various (6): MI.Eliseo Martínez2014-11-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problems: Argument with 'nonnull' attribute passed null @ 277. http://neovim.org/doc/reports/clang/report-9c3614.html#EndPath Result of operation is garbage or undefined @ 281. http://neovim.org/doc/reports/clang/report-45efbf.html#EndPath Argument with 'nonnull' attribute passed null @ 306. http://neovim.org/doc/reports/clang/report-ffb84f.html#EndPath Result of operation is garbage or undefined @ 311. http://neovim.org/doc/reports/clang/report-d04333.html#EndPath Argument with 'nonnull' attribute passed null @ 315. http://neovim.org/doc/reports/clang/report-786819.html#EndPath Uninitialized argument value @ 328. http://neovim.org/doc/reports/clang/report-2a5506.html#EndPath Diagnostic: Multithreading issues. Rationale : All reported problems can only occur if accesed globals change state while executing function, which could only happen in a multithreaded environment. Resolution: Use local variables. Note that this change alters function semantics, as now function only depends on global values at entry time. This shouldn't be a problem, though, as new semantics should be in fact better.
* Fix warnings: cursor_shape.c: parse_shape_opt(): Garbage value: FP.Eliseo Martínez2014-11-06
| | | | | | | | | | | | Problem: Assigned value is garbage or undefined @ 187. http://neovim.org/doc/reports/clang/report-7b7d61.html#EndPath. Diagnostic: False positive. Rationale : `colonp`, must be `>= modep, or null` by `vim_strchr` postcondition. At this point we also it's not null and it's not equal to `modep`, by previous code. So, it must be `> modep`. Resolution: Assert `colonp > modep`.
* Merge PR #1398 'Tests and msgpack rpc improvements'Thiago de Arruda2014-11-06
|\
| * channel: Delay notifications to avoid client race conditionsThiago de Arruda2014-11-06
| | | | | | | | | | | | | | | | | | | | It is currently possible for a client to send a response that doesn't match the current server->client request(at the top of the stack). This commit fixes that by delaying notifications to until the first `channel_send_call` invocation returns. Also remove the "call stack" size check, vim will already break if the call stack goes too deep.
| * test: Add more valgrind "possibly lost" suppressionsThiago de Arruda2014-11-06
| | | | | | | | These valgrind entries are false positives because vim uses interior pointers.
| * test: Simplify/fix options_spec.luaThiago de Arruda2014-11-06
| | | | | | | | | | | | | | | | | | | | The options_spec.lua suite has one purpose: Check if the :options commands will throw any exception(:options is implemented by $VIMRUNTIME/optwin.vim). For this it is best to use the `vim_command` API function since it will automatically catch exceptions and forward them via msgpack-rpc. Also, the option window seems to affect other tests, so call `restart` in the teardown hook.
| * test: Set VIMRUNTIME before running testsThiago de Arruda2014-11-05
|/ | | | This is required for testing scripts under the 'runtime' directory
* Merge pull request #1397 from fwalch/fix-test_fileJohn Szakmeister2014-11-05
|\ | | | | Restore TEST_FILE functionality for running individual tests.
| * CMake: Fix running individual (functional/unit) test.Florian Walch2014-11-05
|/
* Merge pull request #1395 from jszakmeister/fix-run-functional-testsJohn Szakmeister2014-11-05
|\ | | | | build: fix running of functional tests directly with CMake
| * build: fix running of functional tests directly with CMakeJohn Szakmeister2014-11-05
|/ | | | | | | If you aren't just building everything into build/, then the functional tests fail because they can't find the nvim executable. Let's pass in the location of the nvim executable, and set NVIM_PRG environment variable accordingly.
* Merge pull request #1392 from fwalch/fix-build-warningsJohn Szakmeister2014-11-05
|\ | | | | Fix build warnings.
| * CMake: Rename target "test" to "functionaltest".Florian Walch2014-11-05
| | | | | | | | | | | | | | | | Remove build warning: The target name "test" is reserved or not valid for certain CMake features, such as generator expressions, and may result in undefined behavior.
| * CMake: Modify _FORTIFY_SOURCE only for release builds.Florian Walch2014-11-05
| | | | | | | | | | | | Remove build warning for Debug builds: # warning _FORTIFY_SOURCE requires compiling with optimization (-O)
| * CMake: Pass build type CFLAGS to header generator.Florian Walch2014-11-05
|/
* Merge pull request #1390 from jszakmeister/turn-down-fortify-sourceJohn Szakmeister2014-11-05
|\ | | | | build: turn down the _FORTIFY_SOURCE level on gcc >= 4.0
| * build: turn down the _FORTIFY_SOURCE level on gcc >= 4.0John Szakmeister2014-11-05
|/
* Merge PR #1328 'Migrate legacy tests'Thiago de Arruda2014-11-04
|\
| * legacy tests: migrate test21Rainer Borene2014-11-04
| |
| * legacy tests: migrate test7Rainer Borene2014-11-04
| |
| * legacy tests: migrate test6Rainer Borene2014-11-04
| |
| * legacy tests: migrate test28Rainer Borene2014-11-04
| |
| * legacy tests: migrate test20Rainer Borene2014-11-04
| |
| * legacy tests: migrate test97Rainer Borene2014-11-04
| |
| * legacy tests: migrate test46Rainer Borene2014-11-04
| |
| * legacy tests: migrate test27Rainer Borene2014-11-04
| |
| * legacy tests: migrate test54Rainer Borene2014-11-04
| |
| * legacy tests: migrate test41Rainer Borene2014-11-04
| |
| * legacy tests: migrate test23Rainer Borene2014-11-04
| |
| * legacy tests: migrate test56Rainer Borene2014-11-04
| |
| * legacy tests: migrate test_autoformat_joinRainer Borene2014-11-04
| |
| * legacy tests: migrate test_changelistRainer Borene2014-11-04
| |
| * legacy tests: migrate test81Rainer Borene2014-11-04
| |
| * legacy tests: migrate test_utf8Rainer Borene2014-11-04
| |
| * legacy tests: migrate test_insertcountRainer Borene2014-11-04
| |
| * legacy tests: migrate test_optionsRainer Borene2014-11-04
| |
| * legacy tests: migrate test18Rainer Borene2014-11-04
| |
| * legacy tests: migrate test106Rainer Borene2014-11-04
| |
| * legacy tests: migrate test22Rainer Borene2014-11-04
| |
| * legacy tests: migrate test9Rainer Borene2014-11-04
| |
| * legacy tests: migrate test102Rainer Borene2014-11-04
| |
| * legacy tests: migrate test4Rainer Borene2014-11-04
| |
| * legacy tests: improvements to functional helpers.luaRainer Borene2014-11-04
|/ | | | | | - Clean vim registers and functions before each test - Add eval workaround to the request helper method - Export dedent method
* Merge pull request #1287 '[RFC] Add legacy test conversion script.'Thiago de Arruda2014-11-04
|\
| * Add legacy test conversion script.Florian Walch2014-11-04
|/
* Merge pull request #1376 '[RDY] Passing current range to provider calls'Thiago de Arruda2014-11-04
|\
| * Passing current range to provider callshenadzit2014-11-04
|/ | | | - updated script_host_execute and script_host_execute_file to pass current range
* Merge PR #1384 'Add core dump reporting to travis'Thiago de Arruda2014-11-03
|\