aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
* Fix a wrong expectation for vim_FullName() and FullName_save().John Szakmeister2015-05-09
| | | | | | | | | | | | | | It's the second argument, buf, that cannot be NULL. fname is allowed to be NULL. The issue only showed up on the release build when trying to use NULL for fname and the test would segfault unexpectedly (because the NULL check for fname was being optimized out due to the function attributes). FullName_save() also incorrectly assumes that fname cannot be NULL (possibly because of the attribute on vim_FullName), so fix that site as well. This didn't have a corresponding test, so it wasn't visible as test breakage, but did generate a tautological comparison warning in the release build under Clang.
* os/input.c: Eliminate conversion warning from gcc 5 #2617Björn Linse2015-05-09
|
* 'cpoptions': Remove "*" flag #2554David Bürgin2015-05-08
| | | | | | | | | | The "*" flag in 'cpoptions' makes the command :* execute the contents of a register. Removed because 1. the same functionality exists as :@ 2. it hides :* as a useful command-line shortcut for :'<,'> 3. unlike :@ it cannot be used with the * register Helped-by: Michael Reed <m.reed@mykolab.com>
* coverity/62618: fixing "Wrong sizeof argument"Chris Hall2015-05-08
| | | | | | suspicious_sizeof: Passing argument 168UL /* sizeof (ufunc_T) */ * todo to function xmalloc and then casting the return value to ufunc_T ** is suspicious.
* coverity/109019: fixing "Sizeof not portable"Chris Hall2015-05-08
| | | | | | | | | suspicious_sizeof: Passing argument 8UL /* sizeof (char const **) */ to function xcalloc and then casting the return value to char ** is suspicious. In this particular case sizeof (char const **) happens to be equal to sizeof (char const *), but this is not a portable as
* refactor: remove USR_VIMRC_FILE4 #2580Yamakaky2015-05-07
| | | | It was 1) not used and 2) badly treated, compared to USR_VIMRC_FILE3.
* Merge #2470: Remove char_u (5)Eliseo Martínez2015-05-07
|\ | | | | | | | | | | Reviewed-by: Scott Prager <splinterofchaos@gmail.com> Reviewed-by: Michael Reed <m.reed@mykolab.com> Reviewed-by: Eliseo Martínez <eliseomarmol@gmail.com>
| * Update vim_FullName() to define fname as constMark Bainter2015-05-06
| |
| * Make fname const in path_get_absolute_path()Mark Bainter2015-05-06
| | | | | | | | Also a minor change to remove an unneeded cast
| * Refactor modname() to use add_pathsepMark Bainter2015-05-06
| | | | | | | | Based on splinterofchaos review
| * Passing by: wrap mb_ptr_adv(), mb_cptr_adv(), mb_ptr_back() macros in ↵Mark Bainter2015-05-06
| | | | | | | | parenthesis.
| * Merge 'p' declaration with init in vim_version_dir()Mark Bainter2015-05-06
| |
| * Remove char_u: vim_settempdir()Mark Bainter2015-05-06
| |
| * Remove char_u: concat_fnames()Mark Bainter2015-05-06
| |
| * Remove char_u: make_percent_swname()Mark Bainter2015-05-06
| |
| * Remove char_u: fix_fname()Mark Bainter2015-05-06
| |
| * Remove char_u: FullName_save()Mark Bainter2015-05-06
| |
| * Remove char_u: vim_FullName()Mark Bainter2015-05-06
| |
| * Remove char_u: path_with_url()Mark Bainter2015-05-06
| |
| * Remove char_u: path_is_url()Mark Bainter2015-05-06
| |
| * Remove char_u: modname()Mark Bainter2015-05-06
| |
| * Remove char_u: add_pathsep()Mark Bainter2015-05-06
| |
* | vim-patch:7.4.707 #2541Bastian Winkler2015-05-07
| | | | | | | | | | | | | | Problem: Undo files can have their executable bit set. Solution: Strip of the executable bit. (Mikael Berthe) https://github.com/vim/vim/commit/v7-4-707
* | vim-patch:7.4.526 #2562David Bürgin2015-05-07
| | | | | | | | | | | | | | | | | | Problem: matchstr() fails on long text. Daniel Hahler) Solution: Return NFA_TOO_EXPENSIVE from regexec_nl(). (Christian Brabandt) https://github.com/vim/vim/commit/v7-4-526 Helped-by: Michael Reed <m.reed@mykolab.com>
* | vim-patch:7.4.590 #2592Yamakaky2015-05-06
| | | | | | | | | | | | | | | | | | Problem: Using ctrl_x_mode as if it contains flags. Solution: Don't use AND with CTRL_X_OMNI. (Hirohito Higashi) https://github.com/vim/vim/commit/v7-4-590 It was already included in neovim, see #2527.
* | vim-patch:7.4.564 #2591David Bürgin2015-05-05
| | | | | | | | | | | | | | | | | | | | Problem: FEAT_OSFILETYPE is used even though it's never defined. Solution: Remove the code. (Christian Brabandt) https://github.com/vim/vim/commit/v7-4-564 Not applicable, this code has already been removed in Neovim. Included some of the style tweaks.
* | tests: Migrate legacy test 96 #2220David Bürgin2015-05-04
| |
* | 'cpoptions': Remove "H" flag #2556David Bürgin2015-05-04
| |
* | vim-patch:7.4.574 #2539Bastian Winkler2015-05-03
|/ | | | | | | Problem: No error for eval('$'). Solution: Check for empty name. (Yasuhiro Matsumoto) https://github.com/vim/vim/commit/v7-4-574
* unify jobstart, termopen, and system interfacesScott Prager2015-05-02
| | | | | | | | | | | | | | | | | For any of these functions, if {cmd} is a string, execute "&shell &shellcmdflag '{cmd}'", or simply {cmd} if it's a list. In termopen(), if the 'name' option is not supplied, try to guess using '{cmd}' (string) or {cmd}[0] (list). Simplify ex_terminal to use the string form of termopen(). termopen: get name from argument Convert list_to_argv to tv_to_argv. Helped-by: Björn Linse <@bfredl> Helped-by: oni-link <knil.ino@gmail.com> Helped-by: Thiago de Arruda <@tarruda>
* term: use an argument vector for termopen().Scott Prager2015-05-02
| | | | | | | | | | | | | | | | | | | | | | Old behaviour: termopen('cmd') would run `&shell &shcf "cmd"`, which caused the functional tests to fail on some systems due to the process not "owning" the terminal. Also, it is inconsistent with jobstart(). Modify termopen() so that &shell is not invoked, but maintain the old behaviour with :terminal. Factor the common code for building the argument vector from jobstart() and modify the functional tests to call termopen() instead of :terminal (fixes #2354). Also: * Add a 'name' option for termopen() so that `:terminal {cmd}` produces a buffer named "term//{cwd}/{cmd}" and termopen() users can customize the name. * Update the documentation. * Add functional tests for `:terminal` sinse its behaviour now differs from termopen(). Add "test/functional/fixtures/shell-test.c" and move "test/functional/job/tty-test.c" there, too. Helped-by: Justin M. Keyes <@justinmk>
* src: Misc. cleanupMichael Reed2015-05-01
| | | | | These macros (and global) haven't been used since 3baba1e7bc6698e6bc9f1d37fce88b30d6274bc9
* vim-patch:7.4.659 + testMichael Reed2015-05-01
| | | | | | | | | | Problem: When 'ruler' is set the preferred column is reset. (Issue 339) Solution: Don't set curswant when redrawing the status lines. https://code.google.com/p/vim/source/detail?r=v7-4-659 Helped-by: David Bürgin <676c7473@gmail.com> Based on #2347 by @pvinis
* vim-patch:7.4.389Michael Reed2015-05-01
| | | | | | | | | | | | | based on #2429 by @Hettomei Problem: Still sometimes Vim enters Replace mode when starting up. Solution: Use a different solution in detecting the termresponse and location response. (Hayaki Saito) https://code.google.com/p/vim/source/detail?r=v7-4-389 The upstream patch touched a lot of files which don't exist in nvim, hence the small size of this commit.
* Remove obsolete vi options #2553David Bürgin2015-05-01
| | | | | | | | | | | | | | | | | | | | | | These options were never implemented in Vim. They are documented under |missing-options| in runtime/doc/vi_diff.txt: 'autoprint' 'beautify' 'flash' 'graphic' 'hardtabs' 'mesg' 'novice' 'open' 'optimize' 'redraw' 'slowopen' 'sourceany' 'w300' 'w1200' 'w9600' References #2548.
* Merge #2545: Fix clang analysis warnings. (12)Eliseo Martínez2015-05-01
|\ | | | | | | Reviewed-by: oni-link <knil.ino@gmail.com>
| * Fix warnings: ex_docmd.c: ex_tabonly(): Np dereference: FP.Eliseo Martínez2015-05-01
| | | | | | | | | | | | | | | | | | | | | | | | Problem : Dereference of null pointer @ 5816. Diagnostic : False positive. Rationale : first_tabpage is assumed to be NULL after calling goto_tabpage(), which should not be possible (first_tabpage should be not NULL before calling it, and only changed to another valid tab page). Resolution : Assert first_tabpage after calling goto_tabpage(). Helped-by: oni-link <knil.ino@gmail.com>
| * Fix warnings: ex_cmds2.c: ex_listdo(): Np dereference: FP.Eliseo Martínez2015-05-01
| | | | | | | | | | | | | | | | Problem : Dereference of null pointer @ 1909. Diagnostic : False positive. Rationale : Suggested path error first assumes tp == NULL and later one valid_tabpage(tp), which is not possible. Resolution : Assert tp != NULL if valid_tabpage(tp).
| * Fix warnings: ex_cmds2.c: ex_listdo(): Np dereference: FP.Eliseo Martínez2015-05-01
| | | | | | | | | | | | | | | | Problem : Dereference of null pointer @ 1903. Diagnostic : False positive. Rationale : Suggested error path first assumes wp == NULL and later one win_valid(wp), which is not possible. Resolution : Assert wp != NULL if win_valid(wp).
* | Fix warnings: Support '%zu' in vim_snprintf(): Add support.Eliseo Martínez2015-05-01
| |
* | Support '%zu' in vim_snprintf(): Cleanup.Eliseo Martínez2015-05-01
|/ | | | | Some style changes and minimal refactoring, to tidy up code. No behavioural changes intended.
* vim-patch:7.4.706 #2546Bastian Winkler2015-04-30
| | | | | | | | Problem: Window drawn wrong when 'laststatus' is zero and there is a command-line window. (Yclept Nemo) Solution: Set the status height a bit later. (Christian Brabandt) https://github.com/vim/vim/commit/v7-4-706
* Mark some patches re: command-ranges as included #2546Felipe Morales2015-04-30
| | | | reported by @glts at https://github.com/neovim/neovim/pull/2041#issuecomment-97747833
* vim-patch:7.4.653 #2527Florian Walch2015-04-30
| | | | | | | | | Problem: Insert mode completion with complete() may have CTRL-L work like CTRL-P. Solution: Handle completion with complete() differently. (Yasuhiro Matsumoto, Christian Brabandt, Hirohito Higashi) https://github.com/vim/vim/commit/v7-4-653
* jemalloc: Force use of prefixed functions.Florian Walch2015-04-29
| | | | | | | | * Set JEMALLOC_NO_DEMANGLE to be able to use `je_*` functions, regardless of how jemalloc was compiled (--with-jemalloc-prefix) * Show jemalloc information in Neovim's version output. Resolve #2449.
* CMake: Refer to Unibilium instead of LibUnibilium.Florian Walch2015-04-29
|
* vim-patch:7.4.595Felipe Morales2015-04-28
| | | | | | | Problem: The test_command_count test fails when using Japanese. Solution: Force the language to C. (Hirohito Higashi) https://code.google.com/p/vim/source/detail?name=v7-4-595
* vim-patch:7.4.588Felipe Morales2015-04-28
| | | | | | | | Problem: ":0argedit foo" puts the new argument in the second place instead of the first. Solution: Adjust the range type. (Ingo Karkat) https://code.google.com/p/vim/source/detail?name=v7-4-588
* vim-patch:7.4.585Felipe Morales2015-04-28
| | | | | | | Problem: Range for :bdelete does not work. (Ronald Schild) Solution: Also allow unloaded buffers. https://code.google.com/p/vim/source/detail?name=v7-4-585
* vim-patch:7.4.580Felipe Morales2015-04-28
| | | | | | | | Problem: ":52wincmd v" still gives an invalid range error. (Charles Campbell) Solution: Skip over white space. https://code.google.com/p/vim/source/detail?name=v7-4-580