aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | Macro cleanup: USER_HIGHLIGHTMichael Reed2015-03-05
| | |
| * | Macro cleanup: USE_EXE_NAMEMichael Reed2015-03-05
| | |
| * | Macro cleanup: ONE_CLIPBOARDMichael Reed2015-03-05
| | |
| * | Macro cleanup: UNICODE16Michael Reed2015-03-05
| | | | | | | | | | | | Fix clint warnings as well.
| * | Macro cleanup: SPACE_IN_FILENAMEMichael Reed2015-03-05
| | |
| * | Macro cleanup: HAVE_SANDBOXMichael Reed2015-03-05
|/ /
* | vim-patch:7.4.578oni-link2015-03-04
| | | | | | | | | | | | | | | | Problem: Using getcurpos() after "$" in an empty line returns a negative number. Solution: Don't add one when this would overflow. (Hirohito Higashi) https://code.google.com/p/vim/source/detail?r=v7-4-578
* | remote_ui: Fix error messages #2045Carlos Castillo2015-03-04
| | | | | | | | Make the error messages more precise and uniform.
* | jobsend: Don't append extra newline after last itemBjörn Linse2015-03-03
| | | | | | | | This allows sending binary data that is not newline terminated
* | Merge pull request #2083 from fwalch/fix-deps-32John Szakmeister2015-03-02
|\ \ | |/ |/| deps: Fix building 32-bit Lua rocks.
| * deps: Fix building 32-bit Lua rocks.Florian Walch2015-03-02
|/ | | | | | | * Force building Lua rocks instead of trying to install a binary. * Set the CC and LD environment variables to pass CMAKE_C_COMPILER_ARG1 (contains `-m32` for 32-bit builds) to `luarocks build`.
* Merge pull request #2075 from jszakmeister/dont-hardcode-tty-test-pathJohn Szakmeister2015-03-01
|\ | | | | tests: don't hardcode the path to tty-test
| * tests: don't hardcode the path to tty-testJohn Szakmeister2015-02-28
| | | | | | | | | | This fixes the build for those who drive the build directly using CMake.
* | Merge pull request #2079 from jszakmeister/test-fixes-for-bustedJohn Szakmeister2015-03-01
|\ \ | |/ |/| tests: require luassert in the helpers
| * tests: require luassert in the helpersJohn Szakmeister2015-03-01
|/ | | | | | | | This is necessary for newer versions of Busted, otherwise assert will be nil and the tests will die. Note: this does not mean the tests now work with the latest Busted. There are still several issues preventing that from happening.
* deps: Small fix for PR #2072Thiago de Arruda2015-02-28
| | | | | | | A newer version of luafilesystem than the one reported by busted rockspec is required for the unit tests. Also update libvterm sha1/url
* Merge PR #2072 'Add custom target for pinning busted dependencies'Thiago de Arruda2015-02-28
|\
| * deps: Add libvterm to the project dependenciesThiago de Arruda2015-02-28
| | | | | | | | | | libvterm is a terminal emulation library with abstract display. It will be used to implement builtin terminal emulation into Nvim.
| * deps: Add custom target for pinning busted dependenciesThiago de Arruda2015-02-28
|/ | | | | | busted depends on a number of libraries that sometimes introduce crashing bugs. The new custom target makes sure only stable versions of those dependencies are installed.
* test: mark job/pty test as pending #2058Numkil2015-02-26
| | | | | - closes #2057 - see https://github.com/neovim/neovim/issues/2057 for discussion
* illumos requires the use of limits.h for things like INT_MAX #2049Mike Zeller2015-02-26
|
* Fix warnings: message.c: copy_hotkeys_and_msg: Garbage value: FP. #2062Eliseo Martínez2015-02-26
| | | | | | | | | | | | | | Problem : Branch condition evaluates to a garbage value @ 2868. Diagnostic : False positive. Rationale : Array has_hotkey, declared at 2812, is initialized by console_dialog_alloc (only the needed number of elements). That same number of elements is used by copy_hotkeys_and_msg. Suggested path error is impossible, because it involves a different number of elements in those functions. Resolution : Above condition is cumbersome to prove through assertions. Thus, we prefer to just initialize the array to all-false at declaration point before calling console_dialog_alloc.
* Merge pull request #2053 from jszakmeister/fix-libuv-librariesJohn Szakmeister2015-02-25
|\ | | | | Fix collection of libuv dependent libraries.
| * build: filter off libuv from PC_LIBUV_LIBRARIESJohn Szakmeister2015-02-24
| | | | | | | | | | Otherwise, the -rdynamic that is being passed on the command line will require a dynamic link, even though we often want a static one.
| * build: make sure our deps area is picked up for pkg-config filesJohn Szakmeister2015-02-24
|/
* Merge PR #2037 'Job control fixes and pseudo terminal support'Thiago de Arruda2015-02-23
|\
| * job: Allow spawning jobs connected to pseudo terminalsThiago de Arruda2015-02-23
| |
| * job: Refactor process spawning and startup argumentsThiago de Arruda2015-02-23
| | | | | | | | | | | | | | | | - process spawning was decoupled from the rest of the job control logic. The goal is reusing it for spawning processes connected to pseudo terminal file descriptors. - job_start now receives a JobOptions structure containing all the startup options.
| * job: Send SIGTERM before calling job_stop in job_teardownThiago de Arruda2015-02-23
| | | | | | | | | | Send sigterm immediately since it can be caught by processes. If they don't respond and are still alive after a while, SIGKILL will be sent.
| * eval: Fix buffering of data in job autocommandsThiago de Arruda2015-02-23
|/ | | | | Job autocommands will no longer buffer data chunks that don't end in newlines characters.
* Simpler handling for "always-on"/"always-off" options. #2002Florian Walch2015-02-22
| | | | | 'compatible' and 'edcompatible' are forced to be off. 'ttyfast' is forced to be on.
* Merge PR #2039 'Update unibilium for OSX'Thiago de Arruda2015-02-21
|\
| * tui: Also use xterm-256color when TERM=screen and COLORTERM != NULLThiago de Arruda2015-02-21
| | | | | | | | | | When COLORTERM != null force xterm-256color capabilities when TERM equals xterm or screen.
| * deps: Update unibilium version to add OSX supportThiago de Arruda2015-02-21
|/
* Merge #2027: Fix clang analysis warnings. (7)Eliseo Martínez2015-02-21
|\
| * Fix warnings: window.c: close_last_window_tabpage(): Use after free: RI.Eliseo Martínez2015-02-21
| | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Use after free @ 1795. Diagnostic : Real issue. Rationale : prev_curtab can in fact be freed as a result of call `win_close_othertab(win, free_buf, prev_curtab);`, but it's later used at `sprintf(..., tabpage_index(prev_curtab));`. This was introduced at 3ffc5d81c34cfdd535573a50790690c88e4324bb. Resolution : Move prev_idx calculation before the call freeing prev_curtab.
| * Fix warnings: screen.c: screen_puts_len(): Various (4): FP/MI.Eliseo Martínez2015-02-21
|/ | | | | | | | | | | | | | | | Problems : Assigned value is garbage or undefined @ 5363. Result of operation is garbage or undefined @ 5356. Result of operation is garbage or undefined @ 5320. Result of operation is garbage or undefined @ 5192. Diagnostic : False positives / Multithreading issues. Rationale : Suggested error paths contain incoherent values for has_mbyte, enc_utf8, and enc_dbcs, which should always hold the relationship has_mbyte = enc_utf8 || enc_dbcs, with enc_utf8 and enc_dbcs being mutually exclusive. Asserting on the globals, though, fails, because checker believes they could be modified by other threads in between. Resolution : Make local copy of globals and assert relationship on them.
* Merge PR #1997 'Remove term modules'Thiago de Arruda2015-02-21
|\
| * tui: Use unibi_format instead of unibi_runThiago de Arruda2015-02-21
| | | | | | | | | | | | | | | | The current will segfault for large chunks of output because the output buffer will be overrun. Using unibi_format is simple because we can simply flush the buffer when its full.
| * tui: Remove option cacheThiago de Arruda2015-02-21
| |
| * tui: Set setaf/setab capabilities for TERM=xtermThiago de Arruda2015-02-21
| | | | | | | | | | Many common terminals that set TERM=xterm and $COLORTERM support 256 colors. If this is detected, use force the hardcoded xterm's setaf/setab capabilities.
| * refactor: Remove term modules and termcap optionsThiago de Arruda2015-02-21
|/ | | | | | | | | | | | | | | - Removed term.c, term.h and term_defs.h - Tests for T_* values were removed. screen.c was simplified as a consequence(the best strategy for drawing is implemented in the UI layer) - Redraw functions now call ui.c functions directly. Updates are flushed with `ui_flush()` - Removed all termcap options(they now return empty strings for compatibility) - &term/&ttybuiltin options return a constant value(nvim) - &t_Co is still available, but it mirrors t_colors directly - Remove cursor tracking from screen.c and the `screen_start` function. Now the UI is expected to maintain cursor state across any call, and reset it when resized. - Remove unused code
* Merge PR #2032 'Some test fixes'Thiago de Arruda2015-02-21
|\
| * test: Add extra screen expectation to prevent race conditionThiago de Arruda2015-02-20
| | | | | | | | | | This is to ensure ctrl+c is only pressed after the command has started executing in the viml_system_spec.lua system() interrupt test.
| * test: Remove temporary files created by 072_undo_file_spec.luaThiago de Arruda2015-02-20
| |
| * test: Remove -g flag from pynvim call in Screen.debugThiago de Arruda2015-02-20
|/ | | | The flag is no longer valid(pynvim always starts a GUI)
* Merge #1983: Fix coverity issues. (5)Eliseo Martínez2015-02-20
|\
| * coverity/13746: Negative array index write: FP.Eliseo Martínez2015-02-20
| | | | | | | | | | | | | | | | Problem : Negative array index write @ 1042. Diagnostic : False positive. Rationale : Suggested error path cannot occur: idx should be >= 0, as previous check ensures there's a matching while/for. Resolution : Assert idx >= 0.
| * coverity/13683: Out-of-bounds access: RI.Eliseo Martínez2015-02-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Out-of-bounds access @ 3730. Diagnostic : Real issue. Rationale : str is constructed step by step, str_l growing each time. str_m is the maximum length of str. So, at every step, avail is computed to see if the piece to be added fits in. If not, piece is truncated to a max of `avail`, so that str stays in bounds. Such blocks where pieces are added are of the form `if (str_l < str_m)`. It then follows that once one of those pieces exhausts available space on str, no other such block should be entered. Formally: str_l < strl_m && avail = str_m - str_l && x >= avail --> str_l + x >= str_m Now, suggested error path successively enters blocks where str is exhausted. We're not sure if coverity just fails to follow above implications, or, on the contrary, it's aware of them, but it's signaling the more complex possibility of implications not being fulfilled because of possible arithmetic overflows. We opt then to assume this last case, as the possibility is in fact there. Resolution : Refactor code so that tracked condition doesn't depend on arithmetic implications. Check for overflow.
| * coverity/72768: Negative array index read: FP.Eliseo Martínez2015-02-20
|/ | | | | | | | | | | Problem : Negative array index read @ 5674. Diagnostic : False positive. Rationale : Problem occurs if for loop does not find any match, which implies shl->lnum == 0, and then we enter the `if (shl->lnum == lnum)` branch, which implies lnum == 0 as well. That's not possible, as function should not be called with lnum == 0. Resolution : Change conditions `shl->lnum == lnum` into `bot != -1`.
generated by cgit (git 2.34.1) at 2025-06-26 08:23:32 +0000