aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
| * | | | ui: Don't parse abstract_ui codes if there are no attached UIsThiago de Arruda2015-01-12
| | | | |
| * | | | channel: Free channels created from libuv streamsThiago de Arruda2015-01-12
| | | | |
| * | | | ui: Add 'rgb' parameter to ui_attachThiago de Arruda2015-01-12
| | | | | | | | | | | | | | | | | | | | When set to false, nvim will send cterm color numbers with `highlight_set`.
| * | | | ui: Dont resize screen if no UIs are attachedThiago de Arruda2015-01-12
| | | | | | | | | | | | | | | | | | | | | | | | | This prevents a race condition when a UI attaches early in the program and can receive redraw commands for a invalid screen
| * | | | screen: Fix bug that causes separator to always be redraw for utf8Thiago de Arruda2015-01-12
| | | | |
| * | | | screen: Remove abstract_ui test in reset_cterm_colorsThiago de Arruda2015-01-12
| | | | |
| * | | | syntax: Take rgb fg/bg when allocating cterm attr numberThiago de Arruda2015-01-12
| | | | |
| * | | | ui: Merge standout and reverse into one attributeThiago de Arruda2015-01-12
| | | | |
| * | | | input: Ignore invalid "<" key sequencesThiago de Arruda2015-01-12
| | | | | | | | | | | | | | | | | | | | | | | | | Ignoring invalid key sequences simplifies input handling in UIs. The only downside is having to use "<lt>" everytime a "<" is needed on functional tests.
| * | | | ui: Fix ui resizing and change some method namesThiago de Arruda2015-01-12
| | | | |
| * | | | remote_ui: Move handler registration to remote_ui.cThiago de Arruda2015-01-10
| | | | | | | | | | | | | | | | | | | | Also don't defer attach_ui handling
| * | | | ui: Test for abstract_ui whenever a minimal t_colors value is requiredThiago de Arruda2015-01-10
| | | | | | | | | | | | | | | | | | | | | | | | | t_colors should not be checked when abstract_ui is active, because nvim UI is not limited to a terminal.
| * | | | ui: Add mising T_CSV code to abstract_ui termcapThiago de Arruda2015-01-10
| | | | | | | | | | | | | | | | | | | | | | | | | 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-10
| | | | |
| * | | | ui: Add update_fg/update_bg methodsThiago de Arruda2015-01-10
| |/ / / | | | | | | | | | | | | | | | | It is necessary to notify the UI when the default background/foreground colors change in order to render correctly.
* | | | Cleanup: Rename getdigits() family functions.Eliseo Martínez2015-01-11
| | | |
* | | | Cleanup: Refactor variables: long --> int.Eliseo Martínez2015-01-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | At 31c8440fee0bd694093d83af5dee20e3c3dc876c, some variables were changed from int to long, to avoid -Wconversion errors. Long type was the appropiate one because getdigits() was returning a long. Now that we have get_int_digits() and get_long_digits(), we can revert mentioned variables to int, and use get_int_digits() without having -Wconversion warnings.
* | | | Cleanup: Refactor getdigits().Eliseo Martínez2015-01-11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : getdigits() currently returns a long, but at most places, return value is casted (unsafely) into an int. Making casts safe would introduce a lot of fuss in the form of assertions checking for limits. Note : We cannot just change return type to int, because, at some places, legitimate long values are used. For example, in diff.c, for line numbers. Solution : Introduce new functions: - get_digits() : Gets an intmax_t from a string. - get_int_digits() : Wrapper for ints. - get_long_digits() : Wrapper for longs. And replace getdigits() invocations by the appropiate wrapper invocations.
* | | | Cleanup: Refactor option_table_T.number: long --> int.Eliseo Martínez2015-01-11
| | | |
* | | | Cleanup: Refactor prt_{fg,bg,new_bg}col: int --> uint32_t.Eliseo Martínez2015-01-11
| | | |
* | | | Cleanup: Remove local var bg_color@prt_get_attr().Eliseo Martínez2015-01-11
| | | |
* | | | 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