aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* vim-patch:7.4.633 #2526Florian Walch2015-04-28
| | | | | | | Problem: After 7.4.630 the problem persists. Solution: Also skip redo when calling a user function. https://github.com/vim/vim/commit/v7-4-633
* vim-patch:7.4.630 #2524Florian Walch2015-04-28
| | | | | | | | | Problem: When using Insert mode completion combined with autocommands the redo command may not work. Solution: Do not save the redo buffer when executing autocommands. (Yasuhiro Matsumoto) https://github.com/vim/vim/commit/v7-4-630
* coverity/109843: Nesting indent mismatch: RI.Eliseo Martínez2015-04-28
| | | | Introduction of asserts broke bracketless if's.
* Fix visual selection after left click on tablineMarco Hinz2015-04-28
| | | | | | After left clicking on a tab in the tabline, the "in_tab_line" variable wasn't set to 'false' and every following mouse action assumed still being on the tabline which messed up visual selection etc.
* Merge #2516: Fix clang analysis warnings. (11)Eliseo Martínez2015-04-28
|\ | | | | | | Reviewed-by: oni-link <knil.ino@gmail.com>
| * Fix warnings: ex_getln.c: getexmodeline(): HI.Eliseo Martínez2015-04-27
| | | | | | | | | | | | | | | | | | Problem : Dead increment @ 1795. Diagnostic : Harmless issue. Rationale : Line was necessary before (indent was used by the following while loop), but now that loop is removed, assignment is indeed dead. Resolution : Remove line.
| * Fix warnings: ex_getln.c: init_history(): Nonnull attr passed null: HI.Eliseo Martínez2015-04-27
| | | | | | | | | | | | | | | | | | | | | | Problem : Argument with 'nonnull' attribute passed null @ 4227. Diagnostic : Harmless issue. Rationale : It's true we're calling memset with NULL pointer, but it's also true we're doing it with zero size. We just thought that would work and do nothing (it was a way of avoiding to add a guard). It doesn't work, though, as memset requires nonnull arguments. Resolution : Add guard to avoid null argument.
* | Enable -Wconversion: normal.c.Eliseo Martínez2015-04-27
|/ | | | | | | | | | | | | | | | | | | | | | | | Refactor summary: - extern int opcount --> extern long opcount - bool find_decl(..., int len, ...) --> bool find_decl(..., size_t len, ...) * int find_ident_under_cursor(...) --> size_t find_ident_under_cursor(...) - int find_ident_at_pos(...) --> size_t find_ident_at_pos(...) - int modify_fname(..., int *usedlen, ..., int *fnamelen) --> int modify_fname(..., size_t *usedlen, ..., size_t *fnamelen) * char_u *eval_vars(..., int *usedlen, ...) --> char_u *eval_vars(..., size_t *usedlen, ...) - int find_cmdline_var(..., int *usedlen) --> ssize_t find_cmdline_var(..., size_t *usedlen) - static char_u *repl_cmdline(..., int srclen, ...) --> static char_u *repl_cmdline(..., size_t srclen, ...) - bool get_visual_text(..., int *lenp) --> bool get_visual_text(..., size_t *lenp) * char_u *find_file_name_in_path(..., int len, ...) --> char_u *find_file_name_in_path(..., size_t len, ...) - static char_u *eval_includeexpr(..., int len) --> static char_u *eval_includeexpr(..., size_t len) - char_u *find_file_in_path(..., int len, ...) --> char_u *find_file_in_path(..., size_t len, ...) * char_u *find_file_in_path_option(..., int len, ...) --> char_u *find_file_in_path_option(..., size_t len, ...) - char_u *find_directory_in_path(..., int len, ...) --> char_u *find_directory_in_path(..., size_t len, ...) * int spell_move_to(...) --> size_t spell_move_to(...) - int spell_check(...) --> size_t spell_check(...) - static int spell_bad_len --> static size_t spell_bad_len - void find_pattern_in_path(..., int len, ...) --> void find_pattern_in_path(..., size_t len, ...) Helped-by: Justin M. Keyes <justinkz@gmail.com>
* vim-patch:7.4.712David Bürgin2015-04-27
| | | | | | | Problem: Missing change in another file. Solution: Also change message.c https://github.com/vim/vim/releases/tag/v7-4-712
* vim-patch:7.4.711David Bürgin2015-04-27
| | | | | | | Problem: Missing change in one file. Solution: Also change option.c https://github.com/vim/vim/releases/tag/v7-4-711
* vim-patch:7.4.710David Bürgin2015-04-27
| | | | | | | | | Problem: It is not possible to make spaces visibible in list mode. Solution: Add the "space" item to 'listchars'. (David Bürgin, issue 350) https://github.com/vim/vim/releases/tag/v7-4-710 Closes #2485.
* version.c: Add new patch numbersDavid Bürgin2015-04-27
|
* vim-patch:7.4.537 #2509David Bürgin2015-04-27
| | | | | | | | Problem: Value of v:hlsearch reflects an internal variable. Solution: Make the value reflect whether search highlighting is actually displayed. (Christian Brabandt) https://github.com/vim/vim/releases/tag/v7-4-537
* vim-patch:7.4.532 #2504David Bürgin2015-04-27
| | | | | | | | Problem: When using 'incsearch' "2/pattern/e" highlights the first match. Solution: Move the code to set extra_col inside the loop for count. (Ozaki Kiichi) https://github.com/vim/vim/releases/tag/v7-4-532
* vim-patch:7.4.554David Bürgin2015-04-27
| | | | | | | Problem: Missing part of patch 7.4.519. Solution: Copy back regprog after calling vim_regexec. https://github.com/vim/vim/tree/v7-4-554
* vim-patch:7.4.519David Bürgin2015-04-27
| | | | | | | | | Problem: Crash when using syntax highlighting. Solution: When regprog is freed and replaced, store the result. https://github.com/vim/vim/tree/v7-4-519 Helped-by: Scott Prager <splinterofchaos@gmail.com>
* 'cpoptions': Remove "w" flag #2507David Bürgin2015-04-27
|
* vim-patch: mark 7.4.690 as includedoni-link2015-04-26
| | | | vim-patch: https://code.google.com/p/vim/source/detail?r=v7-4-690
* ex_getln: Fixes for some problems with getexmodeline()oni-link2015-04-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problems with line-input editing commands in Ex mode ('nvim -e' or 'Q' in normal mode): CTRL_U: Only after the next keystroke the complete line change is shown if the indentation was not 0. : asdfCTRL_U CTRL_T: Memory reallocation with ga_grows() before a memmove(), but pointers into the memory are not updated. :set sw=40 :CTRL_T ... Should complain about a memory error. CTRL_D: Trying to reduce an indentation of 0. :set sw=1 :CTRL_D Segfault Helped-by: Scott Prager <splinterofchaos@gmail.com>
* spell.c: give priority to rare words. #2456Jakson Alves de Aquino2015-04-25
| | | | | See: https://groups.google.com/forum/#!topic/vim_dev/rPWOoR3ZgSA
* vim-patch:7.4.699 #2508grtlr2015-04-25
| | | | | | | | Problem: E315 when trying to delete a fold. (Yutao Yuan) Solution: Make sure the fold doesn't go beyond the last buffer line. (Christian Brabandt) https://code.google.com/p/vim/source/detail?r=v7-4-699
* vim-patch:7.4.553 #2513David Bürgin2015-04-25
| | | | | | | Problem: Various small issues. Solution: Fix those issues. https://github.com/vim/vim/releases/tag/v7-4-553
* vim-patch:7.4.550 #2510David Bürgin2015-04-25
| | | | | | | | | Problem: curs_rows() function is always called with the second argument false. Solution: Remove the argument. (Christian Brabandt) validate_botline_win() can then also be removed. https://github.com/vim/vim/releases/tag/v7-4-550
* options: Remove "g" flag from 'cpoptions' #2505David Bürgin2015-04-25
|
* Improve comments and fix ascii_* attributesFelipe Oliveira Carvalho2015-04-24
|
* Replace vim_isspace() with ascii_isspace() defined in ascii.hFelipe Oliveira Carvalho2015-04-24
|
* Replace vim_isxdigit() with to ascii_isxdigit() defined in ascii.hFelipe Oliveira Carvalho2015-04-24
|
* Replace VIM_ISDIGIT() and vim_isdigit() with ascii_isdigit() defined in ascii.hFelipe Oliveira Carvalho2015-04-24
|
* Replace vim_iswhite with ascii_iswhite() defined in ascii.hFelipe Oliveira Carvalho2015-04-24
|
* rpcstart: check argvars, /then/ alloc argv #2462Scott Prager2015-04-23
| | | | fixes #2458
* Merge #2491: Fix xfree of static value from vim_getenv() refactor.Eliseo Martínez2015-04-23
|\ | | | | | | Reviewed-by: Eliseo Martínez <eliseomarmol@gmail.com>
| * Fix xfree of static value in expand_shellcmd()Mark Bainter2015-04-23
| | | | | | | | | | | | | | | | The refactoring of vim_getenv() to remove the mustfree arg included reworking calling functions. expand_shellcmd was also using that to track its usage of the variable within the function, resulting in #2487. This change addresses that scenario and cleans up some of the function for style.
* | TAB0 might not be defined in termios.h #2483Rui Abreu Ferreira2015-04-23
| | | | | | | | | | - OpenBSD termios.h does not have TAB0, skip its use if it is not defined
* | Fix warnings: ex_getln.c: init_history(): NP dereference: FP.Eliseo Martínez2015-04-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem : Dereference of null pointer @ 4199. Diagnostic : False positive. Rationale : Suggested error path implies that newlen == 0 and newlen > hislen, which implies hislen < 0, which is impossible (hislen is always between 0 and 10000). Resolution : Refactor code. Asserting oldlen >= 0 should have sufficed, but for some reason analyzer doesn't follow implications in that case. Asserting newlen != 0 || newlen <= hislen works. But the chance has been taken to do extensive refactoring of this function, as it was difficult to understand as it was. As a result of refactoring, assert is not needed anymore, as we don't call clear_hist_entry() in refactored version. Refactor : - Rework algorithm: * Drop guard for OOM case, which can't happen now. * Drop empty/growing/shrinking cases. Simplify to always doing the same. * Perform circular array reordering in all cases (before, it only did when shrinking). * Work in batches through memcpy/memset, instead of one entry at a time, as it did before. - Inline variable declarations. - Replace `ssize_t` by `int`. - Introduce oldlen as entry value of hislen. - Add a lot of comments. Helped-by: Scott Prager <splinterofchaos@gmail.com> Helped-by: oni-link <knil.ino@gmail.com>
* | Fix warnings: ops.c: op_yank_reg(): Dead assignment: HI.Eliseo Martínez2015-04-23
|/ | | | | | | Problem : Dead assignment @ 2480. Diagnostic : Harmless issue. Rationale : It's true `reg` is not used any more after that point. Resolution : Delete line.
* Add if_python3 supportShougo Matsushita2015-04-21
| | | | | Reviewed-by: Michael Reed <m.reed@mykolab.com>, Daniel Hahler <github@thequod.de> Helped-by: Daniel Hahler <github@thequod.de>
* Merge #2418: Remove char_u (4)Eliseo Martínez2015-04-19
|\ | | | | | | | | | | Reviewed-by: Scott Prager <splinterofchaos@gmail.com> Reviewed-by: Michael Reed <m.reed@mykolab.com> Reviewed-by: Eliseo Martínez <eliseomarmol@gmail.com>
| * Refactor after_pathsep() to use const qualified argsMark Bainter2015-04-18
| |
| * Refactor get_env() to respect const qualifierMark Bainter2015-04-18
| | | | | | | | | | | | Without the casts*, the compiler rightly warns about the os_getenv losing the qualifier. This refactor adds a variable to manage this properly, and renames the original variables to increase clarity.
| * Remove char_u: get_mess_lang()Mark Bainter2015-04-18
| |
| * Remove char_u: set_helplang_default()Mark Bainter2015-04-18
| |
| * Remove char_u: vim_setenv()Mark Bainter2015-04-18
| |
| * Remove char_u: vim_getenv()Mark Bainter2015-04-17
| |
| * Remove char_u: after_pathstep()Mark Bainter2015-04-17
| | | | | | | | See: #459
* | Merge pull request #2443 from jszakmeister/fix-more-release-warningsJohn Szakmeister2015-04-18
|\ \ | | | | | | Fix a few uninitialized variable warnings.
| * | Fix an uninitialized variable warning for call_start.Eliseo Martínez2015-04-18
| | |
| * | Fix a couple uninitialized variable warnings in the release build.John Szakmeister2015-04-18
| |/ | | | | | | | | | | Reviewed-by: Eliseo Martínez <eliseomarmol@gmail.com> Reviewed-by: Marco Hinz <mh.codebro@gmail.com> Reviewed-by: Björn Linse <bjorn.linse@gmail.com>
* | clipboard: cleanup `valid_yank_reg` and use it for `:redir`Björn Linse2015-04-17
| | | | | | | | | | | | | | This fixes missing clipboard support for `:redir` Helped-By: Michael Reed <m.reed@mykolab.com> Helped-By: Scott Prager <splinterofchaos@gmail.com>
* | clipboard: simplify handling of of put in visual mode.Björn Linse2015-04-17
| | | | | | | | | | When clipboard=unnamed and put over visual selection, reduces number of provider calls from 6 to 2. Also add test.
* | ops.c: eliminate unneccesary static variable `y_append`Björn Linse2015-04-17
| |