aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* ci: Do not fail csi_clean if there are no files to removeZyX2017-04-04
|
* ci: Force make output coloringZyX2017-04-04
|
* ci: Clean up when restarting single includes testZyX2017-04-04
|
* Merge #6427 from ZyX-I/writefile-allow-omitting-fsyncJustin M. Keyes2017-04-03
|\ | | | | eval: Make writefile() able to disable fsync()
| * ci: Do not mark test as failed if it is previous one which failedZyX2017-04-03
| |
| * fileio: Fix most linter errorsZyX2017-04-03
| | | | | | One error is still kept: buf_write function is too large.
| * tests: Fix testlint errorsZyX2017-04-03
| |
| * functests: Test some :write errorsZyX2017-04-03
| |
| * fileio: Save details about E212 errorZyX2017-04-03
| |
| * eval: Add s flag, use p_fs by default, error out on unknown flagZyX2017-04-03
| |
| * fileio: Refactor msg_add_fname to something which needs no commentsZyX2017-04-03
| |
| * fileio: Clean up IObuff-manipulation messZyX2017-04-03
| |
| * fixup!ZyX2017-04-03
| |
| * message: Do not use IObuff in emsgfZyX2017-04-03
| |
| * fileio: Refactor errmsg handlingZyX2017-04-03
| | | | | | | | | | | | | | | | Adds os_strerror() result to a number of places. Also since I could not track where err\* variables are NULL and where they are not, using macros to make sure that all three variables are set at once. Removes #ifdef UNIX around the use of os_fsync, makes it use os_close in place of close in some places.
| * eval: Make writefile() able to disable fsync()ZyX2017-04-02
|/
* unittests: Make it easier to determine on which _spec line it crashed (#6424)Nikolai Aleksandrovich Pavlov2017-04-02
| | | | | | Benchmarks: Before change: 17.78s user 3.48s system 94% cpu 22.525 total After change: 25.38s user 4.46s system 101% cpu 29.317 total
* Merge #6423 from justinmk/guicursorJustin M. Keyes2017-04-02
|\
| * api/cursor_style_set: mode descriptionsJustin M. Keyes2017-04-01
| |
| * 'guicursor': Empty means "block cursor in all modes".Justin M. Keyes2017-04-01
| | | | | | | | Also: update default 'guicursor' to match the documentation.
| * tui: 'guicursor' colorMatthieu Coudron2017-04-01
| | | | | | | | | | | | For now only supports valid hex colors (does not check for the validity the hex color) when termguicolors is set, otherwise it won't attempt to change the cursor color.
| * tui: 'guicursor' shape #6044Matthieu Coudron2017-04-01
| | | | | | | | Closes #2583
* | Merge pull request #6338 from jamessan/llvm-updateJames McCoy2017-04-01
|\ \ | | | | | | Bump LLVM to 3.9
| * | ci: Bump clang version to 3.9James McCoy2017-03-31
| | |
| * | ci: Use LLVM's trusty repo to match Travis' Ubuntu versionJames McCoy2017-03-31
| | |
* | | Merge #6409 from ZyX-I/separate-clintJustin M. Keyes2017-04-02
|\ \ \ | |_|/ |/| | cmake: Add `clint` target to build Makefile
| * | ci: Do not shift if there are not enough argumentsZyX2017-03-31
| | |
| * | ci: Add test watchdog and tracing for lint testsZyX2017-03-31
| | |
| * | ci: Enable tracingZyX2017-03-31
| | |
| * | ci: Check for exact value of CI_TARGET, not its emptynessZyX2017-03-31
| | |
| * | ci: Add ${NL} variableZyX2017-03-31
| | |
| * | ci: Do not quote MAKE_CMDZyX2017-03-31
| | |
| * | makefile: Make `lint` target depend on `clint`ZyX2017-03-31
| | |
| * | ci: Refactor CI scriptsZyX2017-03-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 1. CI_TARGET now determines which run_${CI_TARGET}.sh script to use. Defaults to `tests`. 2. Build no longer halts on the first failing suit: e.g. if functional tests failed it will continue with unit tests, etc. 3. All ${MAKE_CMD} occurrences moved to `top_make` function, added `build_make` as an alias to `make -C build` (`"${MAKE_CMD}" -C "${BUILD_DIR}"`) which is too verbose. `suite.sh` was copied from powerline (tests/common.sh file), assumes running with POSIX shells (and actually uses dash in powerline). Then some convenience functions were added (run_test and below).
| * | cmake: Also depend on LINT_PRGZyX2017-03-31
| | |
| * | clint: Do not report zero errorsZyX2017-03-31
| | |
| * | cmake: Replace RunLint.cmake with code in src/nvim/CMakeLists.txtZyX2017-03-31
| | | | | | | | | | | | | | | | | | This also removes LINT_FILE environment variable, other then that functionality is kept. It is expected that developers needing partial linting will use `make lint`, touching interesting file before (if not done already by writing to them).
| * | cmake: Allow failing to download small suppress filesZyX2017-03-31
| | |
| * | cmake: Make Download.cmake check for errorsZyX2017-03-31
| | | | | | | | | Copying from third-party/cmake/DownloadAndExtractFile.cmake.
| * | cmake: Use file-specific supprressesZyX2017-03-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `make -C build clint` time is now make -j5 clint 95.29s user 1.86s system 409% cpu 23.751 total *without* downloading anything (much worse if something was not cached, still a bit better then top-level `make clint`). But since without neovim/bot-ci#95 it is downloading each file one-by-one total time with download (download also parallel!) is make -j5 -B clint 99.29s user 2.98s system 258% cpu 39.634 total Top-level makefile still gives make -j5 clint 59.33s user 0.28s system 95% cpu 1:02.41 total
| * | cmake: Add `clint` target to build MakefileZyX2017-03-31
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allows linting only modified files and linting multiple files in parallel. In the current state is rather slow because errors.json is a 6 MiB file and needs to be reparsed each time. Results on my system (6-core): # In build dir, actually parallel make -j5 clint 241.24s user 8.39s system 334% cpu 1:14.74 total # In root, one process make -j5 clint 60.69s user 0.37s system 93% cpu 1:05.19 total In both cases download time included. That is not well for travis (though I would keep travis as-is because new variant will fail before checking all files), but already good enough for regular development: total times are nearly identical and this is the *full* build, further `make -C build clint` will check only modified files.
* | tui: Only enable/disable mouse when there's something to do (#6411)James McCoy2017-04-02
| | | | | | | | | | | | | | If we get a mouse_on/mouse_off event, but the mouse is already in the corresponding state, there's no need to send the event up to the terminal. Closes #4394
* | Merge #6422 from ZyX-I/fix-6420Justin M. Keyes2017-04-01
|\ \ | | | | | | eval,fileio: Omit additional fsync() call
| * | eval: Do not allocate FileDescriptorZyX2017-04-01
| | |
| * | eval,fileio: Omit additional fsync() callZyX2017-04-01
| | | | | | | | | | | | Fixes #6420
* | | 'pastetoggle': support value >1 char (#6421)Matthew Malcomson2017-04-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If we `set pastetoggle=abcde`, and manually type it, then `vgetorpeek()` sees part of the option before it has all been inserted into the typebuffer. To signify this it sets `keylen = KEYLEN_PART_KEY`, but the condition about whether to return the current key from `vgetorpeek()` only checks for `keylen = KEYLEN_PART_MAP`. Add a check for `KEYLEN_PART_KEY` to account for the `'pastetoggle'` option.
* | | Merge #6418 from ZyX-I/better-unittestsJustin M. Keyes2017-04-01
|\ \ \
| * | | unittests: Replace two environment variables with one TRACE_LEVELZyX2017-04-01
| | | |
| * | | unittests: Fix linter errorZyX2017-04-01
| | | |
| * | | unittests: Disable non-C-callsZyX2017-04-01
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Some benchmarks: TRACE_EVERYTHING: 79.45s user 12.68s system 124% cpu 1:13.94 total (default): 30.26s user 5.30s system 89% cpu 39.663 total