aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | Tests for system()/systemlist() when interrupted with CTRL-C.oni-link2015-01-17
| | | | |
| * | | | job.c: Prevent early return from job_wait().oni-link2015-01-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | A blocking call job_wait(job, -1) can only return after job is finished and all handles of job are closed. But hitting CTRL-C makes job_wait() return early while handles can still be open. This can lead to problems with the job/handle callbacks if the caller (of job_wait()) already freed the memory that is used in the job callbacks. To fix this, only return after all handles of the job are closed.
* | | | | Merge pull request #1833 from aktau/formatc-fix-gcc-on-osxJustin M. Keyes2015-01-16
|\ \ \ \ \ | | | | | | | | | | | | test: Correct lfs.attributes() call, and allow unit tests to run with gcc on OSX
| * | | | | test: fix formatc.lua oddity on OSX/gccNicolas Hillegeer2015-01-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The primitive C canonicalizer we use to strip out duplicate header declarations and keep luajit's ffi happy, didn't work properly in this case. What happened is this (in /usr/include/ctype.h): __DARWIN_CTYPE_TOP_inline int isspecial(int _c) { return (__istype(_c, _CTYPE_T)); } Gets preprocessed to something like: __inline int isspecial(int _c) { return (__istype(_c, _CTYPE_T)); } On OSX/gcc. The formatter wasn't recognizing this entire function as something to put on a single line because it naively just checks for "static" or "inline" for that, but not "__inline". This error doesn't occur on OSX/clang. Without looking further into it, I guess that __DARWIN_CTYPE_TOP_inline gets defined to inline on clang, but __inline on gcc, for some reason. This helps issue #1572 along.
| * | | | | test/os/fs: fix call to lfs.attributesNicolas Hillegeer2015-01-16
|/ / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The second argument to lfs.attributes() serves only to select a specific part of the normally returned table. It's not a file open flag (e.g.: as for fopen() in C). Also made the (n)eq checks a bit more idiomatic. Fixes #1831
* | | | | Merge pull request #1803 from elmart/small-fixesJustin M. Keyes2015-01-16
|\ \ \ \ \ | |_|_|_|/ |/| | | | Small fixes.
| * | | | coverity/100248: Operands don't affect result: HI.Eliseo Martínez2015-01-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Operands don't affect results (CONSTANT_EXPRESSION_RESULT). Diagnostic : Harmless issue. Rationale : n >= LONG_MIN, n being intmax_t, is always true for architectures where sizeof(intmax_t) == sizeof(long). Resolution : Add sizes check.
| * | | | Fix bad assert.Eliseo Martínez2015-01-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Assert can fail for legal values. Modulo-arithmetic of unsigned types can make so that n * 100 > n, but n has overflowed. Solution : Use alternative form of expression.
* | | | | Merge pull request #1816 from Pyrohh/macro_cleanupJustin M. Keyes2015-01-15
|\ \ \ \ \ | | | | | | | | | | | | Macro cleanup
| * | | | | Macro cleanup: MiscellaneousMichael Reed2015-01-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These were found with -Wunused-macros. There are many more macros which triggered that warning, but they were primarily part of larger sets of macros so leave them alone.
| * | | | | Macro cleanup: HAS_SWAP_EXISTS_ACTIONMichael Reed2015-01-14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Neovim always ships with all features[0], so this serves no purpose. Besides, this always evaluated to true. [0]: https://github.com/neovim/neovim/wiki/Differences-from-vim
| * | | | | Macro cleanup: PROTOMichael Reed2015-01-14
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | Regarding dict_lookup() in eval.c: both definitions are the same, the only difference being the spacing between the indirection operator and the indentation level.
* | | | | Merge PR #1810 'abstract_ui fixes and improvements(continuation)'Thiago de Arruda2015-01-15
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | deps: Update libtickit URLThiago de Arruda2015-01-15
| | | | |
| * | | | test: Add more functional test to cover new codeThiago de Arruda2015-01-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - emulate gui_running and terminal colors - scrolling/clearing regions - mouse wheel scrolling - setting icon/title - :stop/:suspend - screen resize
| * | | | screen: Fix screen_resize to consider minimum screen sizeThiago de Arruda2015-01-15
| | | | |
| * | | | input: Fix check for mouse coordinatesThiago de Arruda2015-01-15
| | | | | | | | | | | | | | | | | | | | Must check for EOF which will result in row/col being uninitialized.
| * | | | input: Read row/col position when processing mouse wheelThiago de Arruda2015-01-14
| | | | |
| * | | | ui: Implement set_{title,icon}Thiago de Arruda2015-01-14
| | | | |
| * | | | test: Add Screen.debug for inspecting screens of hanging testsThiago de Arruda2015-01-13
| | | | |
| * | | | term: Fix shell_resized for abstract_uiThiago de Arruda2015-01-13
| | | | |
| * | | | screen: Fix highlight reset for abstract_uiThiago de Arruda2015-01-13
| | | | |
| * | | | main: Fix color schemes for abstract_uiThiago de Arruda2015-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Set 't_Co' to 256 at startup. The value can be changed by the user for compatibility with terminals that are less capable. - `has('gui_running')` will return 1 if at least one rgb UI is attached. Even though these changes are hacky, they are necessary to make the transition to the new UI architecture smoother.
| * | | | syntax: Refresh UI when the color scheme changesThiago de Arruda2015-01-13
| | | | |
| * | | | ui: Fix redraw bug caused by race conditions with remote clientsThiago de Arruda2015-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | Before sending a resize command to the UIs, flush the current output buffer to ensure no redraw commands for a screen with invalid size are processed.
| * | | | shell: When executing command, use screen functions to display outputThiago de Arruda2015-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | By calling ui_write directly, the internal screen isn't updated and invalid bytes aren't handled, which breaks the abstract UI model.
| * | | | ui: Use ui_linefeed to handle line breaks correctlyThiago de Arruda2015-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | ui_linefeed will scroll the screen when it becomes full. This can happen when executing external commands.
| * | | | ui: Fix out_flush/ui_write behavior to always flush for abstract_uiThiago de Arruda2015-01-13
| | | | |
| * | | | channel: Make pending_requests a field of the Channel typeThiago de Arruda2015-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | This is required to send redraw notifications while a msgpack-rpc call is being performed by another channel.
| * | | | ui: Fix `:suspend` command for remote UIsThiago de Arruda2015-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | Forward the command to the remote UI and flush immediately. The semantics/handling is UI-specific.
| * | | | ui: Don't parse abstract_ui codes if there are no attached UIsThiago de Arruda2015-01-13
| | | | |
| * | | | channel: Free channels created from libuv streamsThiago de Arruda2015-01-13
| | | | |
| * | | | ui: Add 'rgb' parameter to ui_attachThiago de Arruda2015-01-13
| | | | | | | | | | | | | | | | | | | | When set to false, nvim will send cterm color numbers with `highlight_set`.
| * | | | syntax: Take rgb fg/bg when allocating cterm attr numberThiago de Arruda2015-01-13
| | | | |
| * | | | ui: Merge standout and reverse into one attributeThiago de Arruda2015-01-13
| | | | |
| * | | | input: Ignore invalid "<" key sequencesThiago de Arruda2015-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | Ignoring invalid key sequences simplifies input handling in UIs. The only downside is having to use "<lt>" everytime a "<" is needed on functional tests.
| * | | | test: Remove unnecessary command from highlight_specThiago de Arruda2015-01-13
| | | | |
| * | | | ui: Fix ui resizing and change some method namesThiago de Arruda2015-01-13
| | | | |
| * | | | remote_ui: Move handler registration to remote_ui.cThiago de Arruda2015-01-13
| | | | | | | | | | | | | | | | | | | | Also disable deferral of attach_ui
| * | | | ui: Add mising T_CSV code to abstract_ui termcapThiago de Arruda2015-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | This code is required for screen.c to set a vertical scroll region, which is a great optimization when scrolling vertically split windows.
| * | | | ui: Increase cursor row when text being rendered would cross its limitThiago de Arruda2015-01-13
| | | | |
| * | | | ui: Add update_fg/update_bg methodsThiago de Arruda2015-01-13
| | | | | | | | | | | | | | | | | | | | | | | | | It is necessary to notify the UI when the default background/foreground colors change in order to render correctly.
| * | | | test: Fix clear/eol_clear in screen.luaThiago de Arruda2015-01-13
|/ / / / | | | | | | | | | | | | These functions need to operate on the current the scroll region.
* | | | Merge pull request #1782 from fwalch/small-patchesJustin M. Keyes2015-01-13
|\ \ \ \ | | | | | | | | | | vim-patch: Multiple small patches
| * | | | vim-patch:7.4.563Florian Walch2015-01-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: No test for replacing on a tab in Virtual replace mode. Solution: Add a test. (Elias Diem) https://code.google.com/p/vim/source/detail?r=v7-4-563
| * | | | vim-patch:7.4.567Florian Walch2015-01-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Non-ascii vertical separater characters are always redrawn. Solution: Compare only the one byte that's stored. (Thiago Padilha) https://code.google.com/p/vim/source/detail?r=v7-4-567
| * | | | version.c: Add patch numbers until 7.4.567.Florian Walch2015-01-08
| | | | |
| * | | | vim-patch:7.4.466Florian Walch2015-01-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: CTRL-W } does not open preview window. (Erik Falor) Solution: Don't set g_do_tagpreview for CTRL-W }. https://code.google.com/p/vim/source/detail?r=v7-4-466
* | | | | Merge pull request #1801 from Pyrohh/qnx_cleanupJustin M. Keyes2015-01-13
|\ \ \ \ \ | | | | | | | | | | | | Remove QNX/pterm remnants
| * | | | | Remove QNX/pterm remnantsMichael Reed2015-01-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The function qnx_init() (wrapped in an ifdef in main.c) doesn't even exist.