aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* | | | Cleanup: Fix bad assert.Eliseo Martínez2015-01-11
|/ / / | | | | | | | | | | | | Assert will fail on first loop iteration (in that case, prtpos.bytes_printed is 0).
* | | Merge pull request #1761 from oni-link/speed.up.gcJustin M. Keyes2015-01-10
|\ \ \ | | | | | | | | Speed up garbage collection (Issue 1687).
| * | | Speed up garbage collection.oni-link2014-12-31
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For garbage collection all lists are kept in first_list, a list of all lists. free_unref_items() searches through first_list and removes unreferenced lists from it (by calling list_free(..., FALSE)). But after a list was removed, the search continues from the beginning of first_list (not sure how many lists were really removed and where to continue in first_list). This is not necessary anymore since vim-patch 7.0.135, because a call to list_free(...,FALSE) makes sure, that no other lists (and dictionaries) are freed. So we always know, that the next list in first_list is still valid (allocated or NULL) and can be used to continue the search. Likewise for dictionaries. Original patch by Ariya Mizutani https://groups.google.com/forum/#!searchin/vim_dev/GC/vim_dev/DBYOdHQWvqY/1WH04_dwETIJ
* | | | Remove long_u: term: Remove dead code using long_u.Eliseo Martínez2015-01-10
| | | | | | | | | | | | | | | | | | | | get_long_from_buf() seems not to be used anywhere, and is the only place where instances of long_u remain at term.c.
* | | | Remove long_u: term: Enable -Wconversion.Eliseo Martínez2015-01-10
| | | |
* | | | Remove long_u: hardcopy: Refactor long_u.Eliseo Martínez2015-01-10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - <color_related_stuff>: long_u --> uint32_t Everywhere long_u was used to hold a color value. Color values are supposed to be 32 bits at most. Supported architectures have 32 bits ints, so we could have used plain ints. But this wouldn't be future-proof, and would be wasteful if a future architecture has ints bigger than 32 bits. So, uint32_t is perfect to achieve optimal packing no matter the architecture. - bytes_to_print/bytes_printed: long_u --> size_t Seems like the correct thing, and gets rid of some casts.
* | | | Remove long_u: hardcopy: Enable -Wconversion.Eliseo Martínez2015-01-10
| | | |
* | | | coverity/13763: Out-of-bounds read: FP.Eliseo Martínez2015-01-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Out-of-bound read from a buffer. Diagnostic : False positive. Rationale : nv_max_linear should always be less than nv_cmd_idx size (NV_CMDS_SIZE). Resolution : Assert rationale.
* | | | coverity/13761: Out-of-bounds-write: FP.Eliseo Martínez2015-01-08
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Out-of-bounds-write to a buffer. Diagnostic : False positive. Diagnostic : Suggested error location is under isupper(c) condition, which makes suggested error impossible. Coverity just fails to take isupper() postcondition into account. Resolution : Assert isupper() postcondition.
* | | | coverity/13758: Out-of-bounds read: FP.Eliseo Martínez2015-01-08
| |/ / |/| | | | | | | | | | | | | | | | | | | | | | | Problem : Out-of-bounds read from a buffer. Diagnostic : False positive. Rationale : Suggested error path implies isupper(*str) being true, which makes error vanish. Coverity just fails to take into account isupper() postcondition. Resolution : Assert isupper() postcondition.
* | | eval: fix incorrect refcount in list_append_listBjörn Linse2015-01-07
| | |
* | | Merge pull request #1763 from epaulson10/vim-7.4.551Justin M. Keyes2015-01-06
|\ \ \ | | | | | | | | vim-patch:7.4.551
| * | | vim-patch:7.4.551Erik Paulson2015-01-04
| |/ / | | | | | | | | | | | | | | | | | | Problem: "ygn" may yank too much. (Fritzophrenic) Issue 295. Solution: Check the width of the next match. (Christian Brabandt) https://code.google.com/p/vim/source/detail?r=v7-4-551
* | / Change 'history' default to 50.Florian Walch2015-01-04
| |/ |/| | | | | | | Default as of Vim 7.4.336 (not yet ported). Runtime files already mention this new default value.
* | Merge pull request #1762 from steveno/patch-1Justin M. Keyes2015-01-03
|\ \ | | | | | | Remove unneeded global WantQueryMouse
| * | globals.h: Remove unused global variableSteven Oliver2015-01-02
| |/
* | vim-patch:7.4.559Florian Walch2015-01-03
| | | | | | | | | | | | | | | | Problem: Appending a block in the middle of a tab does not work correctly when virtualedit is set. Solution: Decrement spaces and count, don't reset them. (James McCoy) https://code.google.com/p/vim/source/detail?r=v7-4-559
* | sha: define SHA256_BUFFER_SIZE and SUM_SIZE.Scott Prager2015-01-02
| |
* | sha256.c: Enable -Wconvert and clint.Scott Prager2015-01-02
|/ | | | | | | | * Add sha256.c/h to clint-files.txt. * Add sha256.c to CONV_SRCS in src/nvim/CMakeLists.txt. * Use size_t for lengths and const for read-only parameters. * Use restrict keyword. * Refine #includes.
* Merge pull request #1742 from fwalch/vim-7.4.552Justin M. Keyes2014-12-30
|\ | | | | vim-patch:7.4.552
| * vim-patch:7.4.552Florian Walch2014-12-26
| | | | | | | | | | | | | | Problem: Langmap applies to Insert mode expression mappings. Solution: Check for Insert mode. (Daniel Hahler) https://code.google.com/p/vim/source/detail?r=v7-4-552
* | Merge pull request #1738 from fwalch/vim-7.4.549Justin M. Keyes2014-12-30
|\ \ | | | | | | vim-patch:7.4.549
| * | vim-patch:7.4.549Florian Walch2014-12-27
| | | | | | | | | | | | | | | | | | | | | Problem: Function name not recognized correctly when inside a function. Solution: Don't check for an alpha character. https://code.google.com/p/vim/source/detail?r=v7-4-549
* | | vim-patch:7.4.487Justin M. Keyes2014-12-29
| | | | | | | | | | | | | | | | | | | | | | | | Problem: ":sign jump" may use another window even though the file is already edited in the current window. Solution: First check if the file is in the current window. (James McCoy) https://code.google.com/p/vim/source/detail?r=v7-4-487
* | | Merge pull request #1737 from fwalch/vim-7.4.541Justin M. Keyes2014-12-29
|\ \ \ | | | | | | | | vim-patch:7.4.541
| * | | vim-patch:7.4.541Florian Walch2014-12-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when doing a range assign. Solution: Check for NULL poiter. (Yukihiro Nakadaira) https://code.google.com/p/vim/source/detail?r=v7-4-541
* | | | vim-patch:7.4.533oni-link2014-12-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: ":hardcopy" leaks memory in case of errors. Solution: Free memory in all code paths. (Christian Brabandt) https://code.google.com/p/vim/source/detail?r=v7-4-533
* | | | vim-patch: Mark patches as NAoni-link2014-12-29
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Code modified in patch 7.4.402 and 7.4.443 was introduced with patch 7.4.399, which is also marked as NA (FEAT_CRYPT was removed). Patch 7.4.509 needs the removed FEAT_CRYPT.
* | | | vim-patch:7.4.211oni-link2014-12-28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: ":lu" is an abbreviation for ":lua", but it should be ":lunmap". (ZyX) Solution: Move "lunmap" to above "lua". https://code.google.com/p/vim/source/detail?r=e90bef2240c8d187da6e8d8fa5007ec5afc12284
* | | | restore 'Not Implemented' for legacy script commands (:lua :ruby ...)Justin M. Keyes2014-12-27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - restores behavior equivalent to Vim compiled without FEAT_MZSCHEME, etc. (avoids spurious "E492: Not an editor command: EOF") for code such as: ```vim if 0 perl << EOF end EOF endif ``` see :help script-here - fixes #1308 - fixes https://github.com/junegunn/vim-plug/issues/111
* | | | Merge pull request #1743 from oni-link/vim.patch.7.4.560Justin M. Keyes2014-12-27
|\ \ \ \ | | | | | | | | | | vim-patch: Mark patch 7.4.560 as NA
| * | | | vim-patch: Mark patch 7.4.560 as NAoni-link2014-12-27
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | The memory leak fixed in 7.4.560 was fixed in e10670ac3bccbea707432bdb54e18639cf2e125f. We also do not need to keep `y_current->y_array` as is until done, because our memory allocations cannot fail.
* / | | vim-patch:7.4.528Florian Walch2014-12-27
|/ / / | | | | | | | | | | | | | | | | | | Problem: Crash when using matchadd() (Yasuhiro Matsumoto) Solution: Copy the match regprog. https://code.google.com/p/vim/source/detail?r=v7-4-528
* | | Merge pull request #1735 from fwalch/vim-7.4.527Justin M. Keyes2014-12-26
|\ \ \ | | | | | | | | vim-patch:7.4.527
| * | | vim-patch:7.4.527Florian Walch2014-12-24
| |/ / | | | | | | | | | | | | | | | | | | Problem: Still confusing regexp failure and NFA_TOO_EXPENSIVE. Solution: NFA changes equivalent of 7.4.526. https://code.google.com/p/vim/source/detail?r=v7-4-527
* | | Merge pull request #1734 from fwalch/small-patchesJustin M. Keyes2014-12-26
|\ \ \ | |_|/ |/| | vim-patch: Multiple small patches (4)
| * | vim-patch:7.4.547Florian Walch2014-12-24
| | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Using "vit" does not select a multi-byte character at the end correctly. Solution: Advance the cursor over the multi-byte character. (Christian Brabandt) https://code.google.com/p/vim/source/detail?r=v7-4-547
| * | vim-patch:7.4.545Florian Walch2014-12-24
| | | | | | | | | | | | | | | | | | | | | Problem: Highlighting for multi-line matches is not correct. Solution: Stop highlight at the end of the match. (Hirohito Higashi) https://code.google.com/p/vim/source/detail?r=v7-4-545
| * | vim-patch:7.4.538Florian Walch2014-12-24
| | | | | | | | | | | | | | | | | | | | | | | | Problem: Tests fail with small features plus Python. Solution: Disallow weird combination of options. Do not set "fdm" when folding is disabled. https://code.google.com/p/vim/source/detail?r=v7-4-538
| * | vim-patch:7.4.536Florian Walch2014-12-24
| | | | | | | | | | | | | | | | | | | | | Problem: Test 63 fails when using a black&white terminal. Solution: Add attributes for a non-color terminal. (Christian Brabandt) https://code.google.com/p/vim/source/detail?r=v7-4-536
| * | vim-patch:7.4.521Florian Walch2014-12-24
| | | | | | | | | | | | | | | | | | | | | | | | Problem: When using "vep" a mark is moved to the next line. (Maxi Padulo, Issue 283) Solution: Decrement the line number. (Christian Brabandt) https://code.google.com/p/vim/source/detail?r=v7-4-521
* | | Merge pull request #1684 from justinmk/coverity74717Justin M. Keyes2014-12-26
|\ \ \ | | | | | | | | [RFC] coverity/74717: FP: NULL Pointer Dereference
| * | | coverity/74717: FP: NULL Pointer DereferenceJustin M. Keyes2014-12-23
| | | | | | | | | | | | | | | | | | | | dynamic_buffer_ensure() allocates buf->data; add an assert to make this clear to coverity.
* | | | Merge pull request #1730 from fwalch/vim-7.4.486Justin M. Keyes2014-12-26
|\ \ \ \ | | | | | | | | | | vim-patch:7.4.486
| * | | | vim-patch:7.4.486Florian Walch2014-12-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Check for writing to a yank register is wrong. Solution: Negate the check. (Zyx). Also clean up the #ifdefs. https://code.google.com/p/vim/source/detail?r=v7-4-486
* | | | | Merge pull request #1725 from fwalch/vim-7.4.450Justin M. Keyes2014-12-26
|\ \ \ \ \ | | | | | | | | | | | | vim-patch:7.4.450
| * | | | | vim-patch:7.4.450Florian Walch2014-12-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Not all commands that edit another buffer support the +cmd argument. Solution: Add the +cmd argument to relevant commands. (Marcin Szamotulski) https://code.google.com/p/vim/source/detail?r=v7-4-450
* | | | | | Merge pull request #1718 from jonaias/masterJustin M. Keyes2014-12-26
|\ \ \ \ \ \ | | | | | | | | | | | | | | Transform global variable really_exiting into static.
| * | | | | | Transform global variable really_exiting into static.Jonas Dourado2014-12-26
| | |_|/ / / | |/| | | |
* | | | | | coverity/90712: Remove unnecessary check for NULL.oni-link2014-12-26
| |_|/ / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ** CID 90712: Dereference after null check (FORWARD_NULL) /src/nvim/getchar.c: 3654 in check_abbr() vim_strsave() never returns NULL, so q is never NULL. Removing the NULL check for q should fix this warning.