aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* | | Merge pull request #1618 from elmart/clang-analysis-fixes-5Justin M. Keyes2014-12-06
|\ \ \ | | | | | | | | Fix clang analysis warnings. (5)
| * | | Fix warnings: message.c: msg_attr_keep(): Np dereference: FP.Eliseo Martínez2014-12-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Dereference of null pointer @ 179. Diagnostic : False positive. Rationale : Error occurs if `s` paramater is null, which should not happen. Resolution : Declare parameter as non null.
| * | | Fix warnings: edit.c: ins_compl_next_buf(): Np dereference: MI.Eliseo Martínez2014-12-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Dereference of null pointer @ 3247. Diagnostic : Multithreading issue. Rationale : Problem only occurs if global `ctrl_x_mode` is modified while calling function is executing. Solution : Use local copy instead of global.
* | | | Allow the rest of the old tests to pass with nocp.Julian Mehne2014-12-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | test40.in, test60.in, test_breakindent.in, test_listlbr.in, test_listlbr_utf8.in: To avoid <tab> triggering wildmode, set wildchar to 'cp' default. test_breakindent.ok: Modify expectations to avoid cpoptions+=n
* | | | Allow most old tests to pass with nocp.Joel Teichroeb2014-12-06
|/ / / | | | | | | | | | | | | test69.in: because 'nocp' sets 'esckeys', add an extra <esc> to avoid 'tm' pause.
* | | shell: Fix shell command outputThiago de Arruda2014-12-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | Shell command output was broken in @8a5a8db, which refactored nvim to no longer switch to cooked mode(linefeeds are processed differently). Fix the problem by refactoring write_output to accept to extra arguments that control the flushing behavior and where data will be written to: buffer or directly to the screen.
* | | Remove remaining crypto referencesMichael Reed2014-12-03
| | | | | | | | | | | | | | | | | | | | | Closes https://github.com/neovim/docs/pull/26 Also added stubs for 'cryptmethod' and 'key', and placeholders for explanation regarding removal of crypto functionality.
* | | Merge pull request #1534 from oni-link/fix.leak.detected.in.1510Justin M. Keyes2014-12-03
|\ \ \ | |_|/ |/| | Fix memory leak detected in #1510.
| * | channel.c: Fix for heap-use-after-freeoni-link2014-12-03
| | | | | | | | | | | | | | | | | | | | | | | | | | | ASAN detected this heap-use-after-free. A job started by channel_from_job() could terminate and result in a call to free_channel(), while channel_send_call() was still active/pending and accessing Channel elements. Original patch by @tarruda.
| * | Fix memory leak detected in PR 1510.oni-link2014-12-03
| | | | | | | | | | | | | | | LSAN/ASAN detected, on an error code path, that not all elements of a struct ChannelCallFrame were freed.
* | | shell: Remove kShellOptCooked from ShellOptsThiago de Arruda2014-12-03
| | |
* | | time: Inline microdelay into os_microdelayThiago de Arruda2014-12-03
|/ /
* | Revert "Merge pull request #1587 from equalsraf/tb-env-configs"John Szakmeister2014-12-03
| | | | | | | | | | This reverts commit 53bccaaf99cc9564fe5096d37e9aacfd19e3d2d4, reversing changes made to 4049002f6b1f6db33a497c1f8473ab15233395d5.
* | Merge pull request #1587 from equalsraf/tb-env-configsJohn Szakmeister2014-12-03
|\ \ | | | | | | Don't use env vars for configuration time options
| * | When SANITIZE=ON headers need -DEXITFREERui Abreu Ferreira2014-12-02
| | |
| * | Only use SANITIZE for nvim targetRui Abreu Ferreira2014-12-02
| | | | | | | | | | | | | | | - Clang's Address Sanitizer options may cause problems when running the unit tests, restrict the use of these options to the nvim target.
| * | Don't use env vars for configuration time optionsRui Abreu Ferreira2014-12-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - As a general rule of thumb one shouldn't use environment variables for setting configuration options for CMake. The reason for this is we don't know when CMake will be executed and re-evaluate that variable. - e.g. If we run cmake a first time with a var set, and then run make on a second session (with no var) and cmake is called because a dependency changed, the option would be disabled - This commit removes the use of environment vars from src/nvim/CMakeLists.txt entirely - Removed SKIP_UNITTEST since it could only be used to remove a target at configuration time (and the target was optional anyway) - Turned SANITIZE into an option, clang-asan.sh now passes cmake -DSANITIZE=ON - Removed SKIP_EXEC since it was disabling a target at configuration time (not being used)
* | | Merge pull request #877 from equalsraf/tb-gendeclarations-windowsJohn Szakmeister2014-12-03
|\ \ \ | |_|/ |/| | Refactor declaration generation error detection
| * | Refactor declaration generationRui Abreu Ferreira2014-11-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Call compiler from CMake instead of lua script to generate a preprocessor file - allows for better/early error detection if the compiler fails - Preprocessor files are saved along with the headers as .i files - Accept preprocessor lines with trailing chars after # as is the case in Clang/Windows - The fourth argument to gendeclarations.lua is now the path to the proprocessor output file
* | | Fix newline substitution: Adapt to upstream patch.Eliseo Martínez2014-12-02
| | | | | | | | | | | | | | | Fix previous changes to be compatible with agreed changes to upstream vim (https://code.google.com/p/vim/issues/detail?id=287).
* | | systemlist: add `keepempty` option to preserve final newlineBjörn Linse2014-12-02
| |/ |/|
* | mouse: Remove CHECK_DOUBLE_CLICK conditionalsThiago de Arruda2014-12-02
| | | | | | | | This is now always defined.
* | term: Move more mouse functions to mouse.cThiago de Arruda2014-12-02
| |
* | term: Remove most calls to settmodeThiago de Arruda2014-12-02
| | | | | | | | | | | | Nvim now relies much less on setting terminal mode to cooked mode, remove most calls to settmode, except for those that happen on startup or when suspending. Eventually even those will be handled by the UI layer.
* | term: Move "set_shellsize" to screen.c as "screen_resize"Thiago de Arruda2014-12-02
| |
* | Merge pull request #1582 from LBEaston/FixSegFaultJustin M. Keyes2014-12-01
|\ \ | | | | | | Fix SegFault when entering Command(q:) or Search(q/) History
| * | Fix SegFault when getting full path for filesLBEaston2014-12-01
| | |
* | | Wconversion: fix #1578Justin M. Keyes2014-11-28
|/ /
* | Merge pull request #1497 from splinterofchaos/const-attrJustin M. Keyes2014-11-29
|\ \ | |/ |/| constify and func-attribute memory.c and strings.c
| * strings: Enable -Wconvert.Scott Prager2014-11-27
| |
| * strings/memory: constify and func_attr.Scott Prager2014-11-27
| | | | | | | | Fix MB_COPY_cHAR() to accept const pointers.
* | Merge pull request #1491 from Pyrohh/os2-and-misc-cleanupJustin M. Keyes2014-11-28
|\ \ | | | | | | Remove MiNT, NeXTSTEP, OS/2 References
| * | Remove OS/2 referencesMichael Reed2014-11-27
| | | | | | | | | | | | | | | Paul Slootman was removed from the top of os_unix.c as OS/2 is no longer supported, but is still credited in runtime/doc/intro.txt.
| * | Remove Atari MiNT referencesMichael Reed2014-11-27
| | |
| * | Remove NeXTSTEP referencesMichael Reed2014-11-27
| | | | | | | | | | | | Remove trailing whitespace as well.
* | | Merge pull request #1533 from elmart/fix-sub-newline-abortJustin M. Keyes2014-11-28
|\ \ \ | | | | | | | | Fix newline substitution causing abort
| * | | Fix newline substitution.Eliseo Martínez2014-11-28
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Command `s/\n//` is being translated into a call to do_join with a count of 1. But do_join asserts its precondition count >= 2, which is causing the program to abort. Note : This in fact revealed bigger problems: generated join command line count, as well as reported substitutions/lines were wrong in several cases, since patch 7.4.232. See: [patch] http://markmail.org/message/vo7ruair5raccawp [issue] https://code.google.com/p/vim/issues/detail?id=287 Solution : - Don't generate join command for single-line-range case. - Make generated join command include: * lines in range + 1, when range doesn't end at last line. * lines in range, when range ends at last line. - Make reported substitutions/lines always be number-of-lines-joined - 1.
* | | Merge pull request #1186 from splinterofchaos/writeJustin M. Keyes2014-11-28
|\ \ \ | |/ / |/| | Non-unix-specific os_unix function.
| * | mch_early_init() -> early_init().Scott Prager2014-11-28
| | | | | | | | | | | | | | | Move general initialization functions to early_init, which simplifies test/unit/helpers.lua, which requires all these functions.
| * | update mch_suspendScott Prager2014-11-28
| | | | | | | | | | | | | | | Use the portable uv_kill. Don't bother with archaic BeOS.
| * | mch_init -> term_initScott Prager2014-11-28
| | |
| * | mch_write -> term_writeScott Prager2014-11-28
| | | | | | | | | | | | | | | | | | Switch from POSIX's write() to fwrite(stdout,...) and disable buffering since vim buffers output explicitly and flushes when needed, like when a key is typed.
* | | Merge pull request #1492 from fwalch/fix-wconversionJustin M. Keyes2014-11-27
|\ \ \ | | | | | | | | Fix some more -Wconversion warnings.
| * | | Wconversion: Change long to int64_t in cursor_shape.c, digraph.c.Florian Walch2014-11-27
| | | | | | | | | | | | | | | | Warnings were fixed in #1488.
| * | | Wconversion: Remove regexp_nfa.c from list.Florian Walch2014-11-27
| | | |
| * | | Wconversion: Fix warnings in indent_c.c.Florian Walch2014-11-27
| | | |
| * | | Wconversion: Fix warnings in file_search.c.Florian Walch2014-11-27
| | | |
| * | | Wconversion: Fix warnings in ex_eval.c.Florian Walch2014-11-27
| |/ /
* | | Merge pull request #1566 from splinterofchaos/fix-save-tv-as-stringJustin M. Keyes2014-11-27
|\ \ \ | | | | | | | | eval: Fix coverity false positive.
| * | | eval: Fix coverity false positive.Scott Prager2014-11-27
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | ** CID 74786: Resource leak (RESOURCE_LEAK) /src/nvim/eval.c: 10614 in f_jobsend() /src/nvim/eval.c: 10616 in f_jobsend() save_tv_as_string() should return NULL and input_len <= 0 for an empty string or error. Callers should check that input != NULL instead of input_len > 0 and assert(input == NULL) when the length must be checked.