aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
...
| * | | | | | | | | vim-patch:8.1.1846: inconsistently using GetVimCommand() and v:progpathzeertzjq2022-02-14
| | |_|/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Inconsistently using GetVimCommand() and v:progpath. (Daniel Hahler) Solution: Use GetVimCommand(). (closes vim/vim#4806) https://github.com/vim/vim/commit/93344c2d707d9953f351c944e6a237c9916f69a3 Cherry-pick a change to test_profile.vim from patch 8.1.1544. Cherry-pick a change to test_vimscript.vim from patch 8.1.1826. Some of the args are no-op in Nvim, and `-i NONE` and `--headless` are already added by `GetVimCommand()`. I'll try to match the order of args in upstream, substituting `--not-a-term` with `--headless`.
* | | | | | | | | docs: clarify ftdetect scripts loading during packadd (#17465)Aetf2022-02-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | The old description doesn't match the current behavior anymore.
* | | | | | | | | Merge pull request #17480 from zeertzjq/vim-8.2.4427zeertzjq2022-02-21
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | vim-patch:8.2.4427: getchar() may return modifiers if no character is available
| * | | | | | | | | vim-patch:8.2.4427: getchar() may return modifiers if no character is availablezeertzjq2022-02-21
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: getchar() may return modifiers if no character is available. Solution: Do not process modifiers when there is no character. (closes vim/vim#9806) https://github.com/vim/vim/commit/ad6c45f62558e03d3e3a927b3fe4dbaf30a36bef
* | | | | | / / / fix(diagnostic): use botright copen for qflist (#17475)Michael Lingelbach2022-02-20
| |_|_|_|_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | This matches the LSP handlers, and forces the qflist for diagnostics to span across the horizontal space, below all open windows.
* | | | | | | | fix(lsp): use botright copen for all handlers (#17471)Michael Lingelbach2022-02-20
| | | | | | | |
* | | | | | | | Merge pull request #17469 from jamessan/use-sysdeps-for-lint-ciJames McCoy2022-02-20
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | Use system dependencies for lint CI
| * | | | | | | | ci: only cache third-party deps if they existJames McCoy2022-02-19
| | | | | | | | |
| * | | | | | | | ci: ensure ~/.cache existsJames McCoy2022-02-19
| | | | | | | | |
| * | | | | | | | ci(lint): explicity build nvimJames McCoy2022-02-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | “make clint-full” bypasses the normal mechanisms used to communicate build flags in the CI jobs, so explicitly build nvim before running the lint jobs.
| * | | | | | | | ci(lint): build against system depsJames McCoy2022-02-19
| | | | | | | | |
| * | | | | | | | ci(lint): install deps via apt instead of building from third-partyJames McCoy2022-02-19
| | |_|/ / / / / | |/| | | | | |
* | | | | | | | Merge pull request #17470 from dundargoc/ci/remove-successJames McCoy2022-02-20
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | ci: remove success check since it's the default
| * | | | | | | ci: remove success check since it's the defaultDundar Göc2022-02-20
|/ / / / / / /
* | | | | | | Merge pull request #17460 from seandewar/vim-8.2.4419Sean Dewar2022-02-19
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | vim-patch:8.2.{4403,4418,4419,4422}
| * | | | | | | vim-patch:8.2.4422: autochdir test fails on MS-WindowsSean Dewar2022-02-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Autochdir test fails on MS-Windows. Solution: Expecta nother error on MS-Windows. https://github.com/vim/vim/commit/adbb383e0f2bb59286ea8133f02c448fd334958f
| * | | | | | | vim-patch:8.2.4419: illegal memory access when using 20 highlightsSean Dewar2022-02-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Illegal memory access when using exactly 20 highlights. Solution: Add one more item in the array. (Brandon Richardson, closes vim/vim#9800) https://github.com/vim/vim/commit/a493b6506b67887a1cc2d1c00a896598c3b2d445
| * | | | | | | vim-patch:8.2.4418: crash when using special multi-byte characterSean Dewar2022-02-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when using special multi-byte character. Solution: Don't use isalpha() for an arbitrary character. https://github.com/vim/vim/commit/5921aeb5741fc6e84c870d68c7c35b93ad0c9f87 Rename vim_isalpha to mb_isalpha.
| * | | | | | | vim-patch:8.2.4403: ml_get error with nested folds and deleting linesSean Dewar2022-02-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: ml_get error with nested folds and deleting lines. Solution: Correct the last line number before calling hasFoldingWin(). https://github.com/vim/vim/commit/943773783384a5ff63f57769d37ddabf8156fe1e
* | | | | | | | vim-patch:8.2.4424: ".gts" and ".gjs" files are not recognized (#17464)Christian Clason2022-02-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: ".gts" and ".gjs" files are not recognized. Solution: Recognize Glimmer flavored typescript and javascript. (closes vim/vim#9799) https://github.com/vim/vim/commit/cdf717283ca70b18f20b8a2cefe7957083280c6f
* | | | | | | | fix: lsp and diagnostic highlight priority (#17461)Michael Lingelbach2022-02-19
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes https://github.com/neovim/neovim/issues/17456 * treesitter uses the default highlight priority of 50 * diagnostic highlights have a priority of 150 * lsp reference highlights have a priority of 200 This ensures proper ordering.
* | | | | | | Merge pull request #17454 from dundargoc/ci/labeler/testJames McCoy2022-02-18
|\ \ \ \ \ \ \ | | |_|_|_|_|/ | |/| | | | | ci(labeler): change "tests" to "test"
| * | | | | | ci(labeler): change "tests" to "test"Dundar Göc2022-02-18
|/ / / / / /
* | | | | | vim-patch:8.2.4414: solidity files are not recognized (#17451)Christian Clason2022-02-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Solidity files are not recognized. Solution: Add the *.sol pattern. (Dundar Goc, closes vim/vim#9792) https://github.com/vim/vim/commit/97b231541d4e82fbc85e51121448d95bd43c50ad
* | | | | | Merge pull request #17200 from lewis6991/_loadfilebfredl2022-02-18
|\ \ \ \ \ \ | | | | | | | | | | | | | | refactor(lua): call `loadfile` internally instead of `luaL_loadfile`
| * | | | | | refactor(lua): call loadfile internallyLewis Russell2022-02-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | .. instead of luaL_loadfile allows files to be cached
* | | | | | | Merge pull request #17445 from dundargoc/ci/review/use-checkoutJames McCoy2022-02-18
|\ \ \ \ \ \ \
| * | | | | | | ci: download reviews.js through checkout instead of wgetDundar Göc2022-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes testing the workflows much smoother.
* | | | | | | | Merge pull request #17404 from dundargoc/ci/put-each-test-in-separate-stepJames McCoy2022-02-18
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | ci: run each test suite in a separate github step
| * | | | | | | | ci: run each test suite in a separate github stepDundar Göc2022-02-18
| | |/ / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This should help combat some of the lagginess when looking at the CI logs in the browser.
* | | | | | | | Merge pull request #17450 from dundargoc/ci/lint/fix-keyJames McCoy2022-02-18
|\ \ \ \ \ \ \ \ | |/ / / / / / / |/| | | | | | | ci(lint): simplify cache key
| * | | | | | | ci(lint): simplify cache keyDundar Göc2022-02-18
|/ / / / / / /
* | | | | | | Merge pull request #17449 from zeertzjq/vim-8.2.3659zeertzjq2022-02-18
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | vim-patch:8.2.{3659,3660,3661}: integer overflow with large line number
| * | | | | | | vim-patch:8.2.3661: test for put with large count failszeertzjq2022-02-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Test for put with large count fails. Solution: Adjust the counts in the test. https://github.com/vim/vim/commit/8bc07e800c2af36686aadd4178cc2671f5c454d4
| * | | | | | | vim-patch:8.2.3660: overflow check uses wrong numberzeertzjq2022-02-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Overflow check uses wrong number. Solution: Divide by ten. https://github.com/vim/vim/commit/9b0e82f35ed4e98414333e71b71ca56219683d16
| * | | | | | | vim-patch:8.2.3659: integer overflow with large line numberzeertzjq2022-02-18
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Integer overflow with large line number. Solution: Check for overflow. (closes vim/vim#9202) https://github.com/vim/vim/commit/03725c5795ae5b8c14da4a39cd0ce723c6dd4304 Put E1247 in globals.h as E1240 is also there. Do not make getdigits() abort.
* | | | | | | Merge pull request #17433 from seandewar/vim-8.2.3492Sean Dewar2022-02-18
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | | vim-patch:8.2.{3492,3493,3570,3573,3574,3575,3577,3601}: put overflow checking shenanigans
| * | | | | | vim-patch:8.2.3601: check for overflow in put count does not work wellSean Dewar2022-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Check for overflow in put count does not work well. Solution: Improve the overflow check. (Ozaki Kiichi, closes vim/vim#9102) https://github.com/vim/vim/commit/fa53722367c3793fda95dac665af74b8651065e9 Add some casts as Nvim uses size_t variables in some places. We could technically adjust the logic to check for overflow outside of size_t's range, but it's much easier to just port the patch exactly (also means we can use the same tests). v:sizeoflong is N/A, so convert the 64-bit tests to Lua and use the FFI to check long's size.
| * | | | | | fix(ops): str_to_reg passing NULL to memcpySean Dewar2022-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Required for the tests introduced in v8.2.3601 to pass ASAN when running test_alot.vim. Co-authored-by: erw7 <erw7.github@gmail.com>
| * | | | | | vim-patch:8.2.3577: overflow check fails with 32 intsSean Dewar2022-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Overflow check fails with 32 ints. Solution: Only test with 64 bit ints. https://github.com/vim/vim/commit/0f0044125c2a5dcde2c4605efc39d2e237eed024
| * | | | | | vim-patch:8.2.3575: overflow check still fails when sizeof(int) == sizeof(long)Sean Dewar2022-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Overflow check still fails when sizeof(int) == sizeof(long). Solution: Use a float to check the result. https://github.com/vim/vim/commit/e551ccfb9311eea5252d1c3106ff7a53c762d994 This approach is... interesting... Tests fail.
| * | | | | | vim-patch:8.2.3574: divide by zeroSean Dewar2022-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Divide by zero. Solution: Don't check for overflow if multiplicand is zero. https://github.com/vim/vim/commit/8a1962d1355096af55e84b1ea2f0baf5f1c5a5bc
| * | | | | | vim-patch:8.2.3573: cannot decide whether to skip test that fails with 64 bitSean Dewar2022-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Cannot decide whether to skip test that fails with 64 bit ints. (closes vim/vim#9072) Solution: Add v:sizeofint, v:sizeoflong and v:sizeofpointer. Improve the check for multiply overflow. https://github.com/vim/vim/commit/69b3072d984480935ec412b32b97fea974d2b689 Omit v:sizeof{int,long,pointer} as they're only really used for tests.
| * | | | | | vim-patch:8.2.3570: Test_very_large_count fails on 32bit systemsSean Dewar2022-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Test_very_large_count fails on 32bit systems. Solution: Bail out when using 32 bit numbers. (closes vim/vim#9072) https://github.com/vim/vim/commit/ec6e63079dde24a1d74b4103775e74d00f9215ec
| * | | | | | vim-patch:8.2.3493: large count test fails on MS-WindowsSean Dewar2022-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Large count test fails on MS-Windows. Solution: Skip the test on MS-Windows. https://github.com/vim/vim/commit/cddd5ac911707034ca27f10037c4b1b523188c47
| * | | | | | vim-patch:8.2.3492: crash when pasting too many timesSean Dewar2022-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when pasting too many times. Solution: Limit the size to what fits in an int. (closes vim/vim#8962) https://github.com/vim/vim/commit/eeed1c7ae090c17f4df51cf97b2a9e4d8b4f4dc7 Note that this overflow check pretty bad. It also doesn't work well on Windows (where sizeof(int) == sizeof(long)). This is all temporary; everything here is rewritten in future patches anyway. e_resulting_text_too_long was already cherry-picked. totlen is size_t in Nvim, but is int in Vim. This means we'll need some casts. We could technically adjust the logic in do_put to use the entire range of size_t in stuff like totlen, but there's not much gain, and it's much easier to just port the patch like Vim as was done before (also allows us to use the same tests).
* | | | | | | Merge pull request #17430 from dundargoc/test/remove-travisJames McCoy2022-02-17
|\ \ \ \ \ \ \
| * | | | | | | test: remove checks to see if current CI job is travis or appveyorDundar Göc2022-02-17
| | | | | | | |
* | | | | | | | vim-patch:8.2.4411: bicep files are not recognized (#17447)Christian Clason2022-02-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Bicep files are not recognized. Solution: Match *.bicep files. (Dundar Goc, closes vim/vim#9791) https://github.com/vim/vim/commit/8e5ba693ad9377fbf4b047093624248b81eac854
* | | | | | | | Merge pull request #17427 from dundargoc/ci/lint/remove-unnecessary-stepsJames McCoy2022-02-17
|\ \ \ \ \ \ \ \