aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
* | | | Merge pull request #15575 from dundargoc/refactor/uncrustifyBjörn Linse2021-09-07
|\ \ \ \ | |/ / / |/| | | refactor: format files with uncrustify
| * | | refactor: format files with uncrustifyDundar Göc2021-09-07
|/ / /
* | | fix(screen): missing search highlights when redrawing from timer #15380Jit2021-09-06
| | | | | | | | | | | | | | | | | | | | | * Revert "vim-patch:8.1.2294: cursor pos wrong with concealing and search causes a scroll" * Add a test which covers #13074 910bbc3cca796f7fa941e0f6176cd0061de0e01c while reverting the screen.c code changes from there. Fixes #14064
* | | fix(lsp): adapt codelens resolve to handler signature change (#15578)Mathias Fußenegger2021-09-06
| | | | | | | | | Follow up to https://github.com/neovim/neovim/pull/15504
* | | fix(lsp): update workspace/applyEdit handler signature (#15573)Jose Alvarez2021-09-05
| | |
* | | Merge pull request #15504 from mjlbach/feat/change-handler-signatureMichael Lingelbach2021-09-05
|\ \ \ | | | | | | | | feat(lsp)!: change handler signature
| * | | docs: regenerateMichael Lingelbach2021-09-05
| | | |
| * | | feat(lsp)!: change handler signatureMichael Lingelbach2021-09-05
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously, the handler signature was: function(err, method, params, client_id, bufnr, config) In order to better support external plugins that wish to extend the protocol, there is other information which would be advantageous to forward to the client, such as the original params of the request that generated the callback. In order to do this, we would need to break symmetry of the handlers, to add an additional "params" as the 7th argument. Instead, this PR changes the signature of the handlers to: function(err, result, ctx, config) where ctx (the context) includes params, client_id, and bufnr. This also leaves flexibility for future use-cases. BREAKING_CHANGE: changes the signature of the built-in client handlers, requiring updating handler calls
* | | | Merge pull request #15569 from bfredl/end_fillBjörn Linse2021-09-05
|\ \ \ \ | | | | | | | | | | refactor(screen): let win_line() always handle fillers after last line
| * | | | refactor(screen): let win_line() always handle fillers after last lineBjörn Linse2021-09-05
| | | | |
* | | | | build: fix fpclassify -Wfloat-conversion warning #15570Ben Noordhuis2021-09-05
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Work around a glibc bug where it truncates the argument to fpclassify() from double to float by implementing fpclassify() ourselves. Correctness test (Note that the FP_SUBNORMAL test depends on an atof() that knows how to parse subnormals. Glibc does, not sure about other libcs.): #include <math.h> #include <stdint.h> #include <string.h> int xfpclassify(double d) { uint64_t m; int e; memcpy(&m, &d, sizeof(m)); e = 0x7ff & (m >> 52); m = 0xfffffffffffffULL & m; switch (e) { default: return FP_NORMAL; case 0x000: return m ? FP_SUBNORMAL : FP_ZERO; case 0x7ff: return m ? FP_NAN : FP_INFINITE; } } #include <assert.h> #include <stdlib.h> int main(void) { assert(FP_ZERO == xfpclassify(atof("0.0"))); assert(FP_ZERO == xfpclassify(atof("-0.0"))); assert(FP_NORMAL == xfpclassify(atof("1.0"))); assert(FP_NORMAL == xfpclassify(atof("-1.0"))); assert(FP_INFINITE == xfpclassify(atof("inf"))); assert(FP_INFINITE == xfpclassify(atof("-inf"))); assert(FP_NAN == xfpclassify(atof("nan"))); assert(FP_NAN == xfpclassify(atof("-nan"))); assert(FP_SUBNORMAL == xfpclassify(atof("1.8011670033376514e-308"))); return 0; }
* | | | vim-patch:90df4b9 (#15494)Izhak Jakov2021-09-04
| | | | | | | | | | | | | | | | | | | | Add JSONC runtime files Co-authored-by: Izhak Jakov <izhakjakov>
* | | | Merge pull request #15563 from dundargoc/refactor/uncrustifyBjörn Linse2021-09-04
|\ \ \ \ | | | | | | | | | | refactor: update uncrustify config and format screen.c
| * | | | refactor: update uncrustify config and format screen.cDundar Göc2021-09-03
|/ / / / | | | | | | | | | | | | | | | | Also set new option cmt_trailing_single_line_c_to_cpp to true. It converts trailing, single-line c-comments (/**/) into cpp-comments (//).
* | | | fix(defaults): "syntax sync maxlines=1" on CmdwinEnter #15552Justin M. Keyes2021-09-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | I mistakenly suggested maxlines=&cmdwinheight, forgetting that it is calculated from topline, not cursor. maxlines=1 makes the most sense in cmdwin. ref #15401 622a36b1f1c652a8de433028bc4a03a1216db23f
* | | | feat(defaults): limit syntax cost on CmdwinEnter #15401Gregory Anders2021-09-02
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a new default autocommand to limit syntax highlighting synchronization in the command window. This refactors the nvim_terminal autocommand out of main() and into a new init_default_autocmds() function, which is now part of the startup process and can be further extended with more default autocommands down the road. ref #6289 #6399
* | | | refactor(tests): use assert_alive() #15546Justin M. Keyes2021-09-01
| | | |
* | | | feat(api): nvim_get_chan_info: include "argv" for jobs #15537Justin M. Keyes2021-09-01
| | | | | | | | | | | | ref #15440
* | | | docs(lsp): remove private lsp.diagnostic functions from docs (#15541)Mathias Fußenegger2021-09-01
| | | | | | | | | | | | | | | | Both `apply_to_diagnostic_items` and `show_diagnostics` are local functions and cannot be called by users.
* | | | docs(lsp): document codelens.get bufnr parameter (#15540)Mathias Fußenegger2021-09-01
| |_|/ |/| | | | | Alternative to https://github.com/neovim/neovim/pull/15224
* | | Merge #15402 fix(terminal): close without ! if the job is stoppedJustin M. Keyes2021-08-31
|\ \ \
| * | | fix(jobwait): always drain process event queues #15402Gregory Anders2021-08-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: jobwait() returns early if the job was stopped, but the job might have pending callbacks on its event queue which are required to complete its teardown. State such as term->closed might not be updated yet (by the pending callbacks), so codepaths such as :bdelete think the job is still running. Solution: Always flush the job's event queue before returning from jobwait(). ref #15349
| * | | fix(terminal): close without ! if the job is stoppedGregory Anders2021-08-31
|/ / / | | | | | | | | | | | | | | | - If the terminal job is still running then ! is still required. Closes #4683
* | | Merge pull request #15526 from bfredl/f_metaBjörn Linse2021-08-30
|\ \ \ | | | | | | | | fix(lua): make core vim module not dependent on $VIMRUNTIME modules
| * | | fix(lua): make core vim module not dependent on $VIMRUNTIME functionsBjörn Linse2021-08-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | fixes #15524 Note: this is obviously a quickfix. A scalabe solution will involve being able to specify a _list_ of modules to be put into packages.preload, without needing to manually copypasta a blurb of C code. Perhaps even involving bytecode for static builds (to speedup initialization)
* | | | fix(tutor): formatting, layout #15098Oliver Marriott2021-08-30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * fix(tutor): adjust over-80ch lines and corresponding expect file * fix(tutor): standardise indentation and formatting, add nowrap modeline - unifies the formatting/layout, which was a bit inconsistent, - adds a nowrap modeline Since the tutor uses a lot of conceals, which are included in the character count when calculating line wrapping, lines were breaking at what looked like odd spots, which gives a poor first impression and lowered readability. I have adjusted some lines to be over 80ch in the source, but once they're rendered out with conceals, they're actually under 80, so even with nowrap we don't visually extend past 80. fix #15088
* | | | fix(lsp): resolve bufnr in buf_is_attached (#15523)Jose Alvarez2021-08-30
|/ / /
* | | chore(flake): fix for recent nixpkgs (#15520)Matthieu Coudron2021-08-30
| | | | | | | | | | | | | | | https://github.com/NixOS/nixpkgs/pull/134463 made keepDebugInfo obsolete for generic packages. This copies what keepDebugInfo used to do.
* | | Merge pull request #15498 from neovim/marvim/api-doc-update/masterThomas Vigouroux2021-08-30
|\ \ \ | | | | | | | | docs: regenerate
| * | | docs: regeneratemarvim2021-08-29
| | | |
* | | | vim-patch:8.2.2938: after using motion force from feedkeys() it sticks (#15240)Xiao2021-08-29
|/ / / | | | | | | | | | | | | Problem: After using motion force from feedkeys() it may not be reset. Solution: Clear motion_force in clearop(). (closes vim/vim#8323) https://github.com/vim/vim/commit/21492743e80c6740bac65a91311c28bede8ef2f8
* | | Merge pull request #15304 from bfredl/quantumtheoryBjörn Linse2021-08-29
|\ \ \ | | | | | | | | fix(lua): preserve argument list ... which has not the same length as #{...}
| * | | fix(lua): preserve argument lists which are not listsBjörn Linse2021-08-29
| | | |
* | | | docs: .git-blame-ignore-revs #15510Justin M. Keyes2021-08-28
|/ / /
* | | Merge pull request #15509 from bfredl/miniperfBjörn Linse2021-08-28
|\ \ \ | | | | | | | | perf(api): avoid spurious allocations when converting small objects
| * | | perf(api): avoid spurious allocations when converting small objectsBjörn Linse2021-08-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Converter functions use a heap-allocated stack to handle complex nested objects. However, these are often called with simple, primitive values like integers or bools wrapped in an Object. Avoid the memory allocation in this case using kvec_withinit_t
| * | | refactor(lua): rename nlua_msgpack_ => nlua_api_Björn Linse2021-08-28
| | | | | | | | | | | | | | | | | | | | | | | | These functions do not involve msgpack. Initially the nvim api was sometimes called the "msgpack API", but entry points from vim script and lua are equally valid (and don't need to reference "msgpack")
* | | | Merge pull request #15465 from dundargoc/refactor/uncrustifyBjörn Linse2021-08-28
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | refactor(codebase): Update uncrustify config and apply it on one file screen.c The plan is to go through the codebase in batches, and remove the most egregious "make lint" backlog failures, by applying this config.
| * | | | refactor: format screen.c with uncrustifyDundar Göc2021-08-28
| | | | |
| * | | | refactor: update uncrustify config file to better fit neovim style guideDundar Göc2021-08-28
|/ / / /
* / / / fix(lsp): check if buffer is valid in changetracking (#15505)Jose Alvarez2021-08-28
|/ / /
* | | fix(process_wait): drain proc.events directly #15501Gregory Anders2021-08-27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | After a process's refcnt is decremented to zero, it enqueues a `process_close_event` on its own event queue. In `process_wait`, this event should be processed immediately so that any process close callbacks are executed before `process_wait` returns. Update `process_wait` to always process the process's event queue after the process is freed, rather than the event queue passed in as an argument.
* | | Merge #15433 defaults: auto-create backup dirJustin M. Keyes2021-08-27
|\ \ \
| * | | docs: update 'backupdir' and 'undodir' descriptionsGregory Anders2021-08-27
| | | |
| * | | fix: remove trailing slashes before making directoryGregory Anders2021-08-27
| | | | | | | | | | | | | | | | | | | | | | | | Remove the trailing slashes from 'undofile' and 'backupdir' before creating directories. They cause problems on Windows which doesn't recognize these slashes as proper path separators.
| * | | feat: defaults: auto-create backup dirGregory Anders2021-08-27
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Copy the behavior of 'undodir' and create the last specified directory in the 'backupdir' option if it doesn't exist. Use trailing slashes for 'backupdir' as well as 'viewdir' and 'undodir' by default. Note that 'undodir' always behaves as though it has the trailing slashes, regardless of whether or not they are present. They are added to the default option value to minimize surprise. The '.' value in 'backupdir' is kept because the default behavior for backups is solely to have a backup if the save of the main file to disk fails. As soon as that save is completed the backup file is removed, so generally there is no need to put them in a central location. Co-authored by: murphy66 <murphy66@gmail.com>
* | | vim-patch:8.1.2229: color number column above/below cursor #15409zeertzjq2021-08-27
| | | | | | | | | | | | | | | Problem: Cannot color number column above/below cursor differently. Solution: Add LineNrAbove and LineNrBelow. (Shaun Brady, closes vim/vim#624) https://github.com/vim/vim/commit/efae76ab1a43d5a628d8c2fa4218ace6ba597f5d
* | | tests(lua/on_yank): assert conditions that fail correctly #15495notomo2021-08-27
| | | | | | | | | | | | The test added in 274a3504a790a799b28ee89c75e29fb4dbdff41f does not fail if the code changes are reverted.
* | | feat(lsp): get_border_size(): support repeating border char list #15474zeertzjq2021-08-27
| | |
* | | fix(lua): verify buffer in highlight.on_yank (#15482)notomo2021-08-26
| | | | | | | | | | | | | | | Resolve an issue with deferred clearing of highlight failing if the buffer is deleted before the timeout by checking whether the buffer is valid first.