aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
...
| * | | | | | | quickfix: Remove unneeded conditionZyX2017-04-16
| | | | | | | | | | | | | | | | | | | | | | | | fmt_ptr was checked for being NULL in if() condition earlier.
| * | | | | | | normal: Add figure braces so that code is clearerZyX2017-04-16
| | | | | | | |
| * | | | | | | normal: Clarify the codeZyX2017-04-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Current variant works only because of PUT_FIXINDENT being equal to true.
| * | | | | | | normal: Remove unneeded assignmentZyX2017-04-16
| | | | | | | |
| * | | | | | | ops: Silence “counter not used in loop” false positiveZyX2017-04-16
| | | | | | | |
| * | | | | | | os/env: Fix “invalid pointer to local” false positiveZyX2017-04-16
| | | | | | | |
| * | | | | | | fileio: Silence “!= identical subexpressions” warningZyX2017-04-16
| | | | | | | |
| * | | | | | | message: Some more has_mbyte/enc_utf8 removalZyX2017-04-16
| | | | | | | |
| * | | | | | | message: Remove some enc_utf8/… checksZyX2017-04-16
| | | | | | | |
| * | | | | | | memline: Fix “NULL pointer dereference” warningZyX2017-04-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It was actually a false positive indicating always-true condition, not real dereference.
| * | | | | | | memline: Silence “buffer underflow” warning, looks like false positiveZyX2017-04-16
| | | | | | | |
| * | | | | | | fileio: Simlify help files encoding detectionZyX2017-04-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Most of code is dead when enc_utf8 is always true. Given that `c` is being reused for other purposes I left it set to 1 just in case.
| * | | | | | | hardcopy: Remove unneeded prt_do_conv assignmentZyX2017-04-16
| | | | | | | |
| * | | | | | | getchar: Fix if block indentationZyX2017-04-16
| | | | | | | |
| * | | | | | | ex_docmd: Remove unneeded if()ZyX2017-04-16
| | | | | | | |
| * | | | | | | ex_docmd: Remove :Ni! easter eggZyX2017-04-16
| | | | | | | |
| * | | | | | | ex_docmd: Remove excessive assignmentZyX2017-04-16
| | | | | | | |
| * | | | | | | farsi: Simplify conditionZyX2017-04-16
| | | | | | | |
| * | | | | | | eval: Fix position of buf declarationZyX2017-04-16
| | | | | | | |
| * | | | | | | eval: Remove unneeded !eap->skip checkZyX2017-04-16
| | | | | | | | | | | | | | | | | | | | | | | | Already checked in the outer if().
| * | | | | | | eval: Remove unneeded varp checkZyX2017-04-16
| | | | | | | |
| * | | | | | | eval: Remove unneeded varp checkZyX2017-04-16
| | | | | | | |
| * | | | | | | eval: Fix condition in f_serverstopZyX2017-04-16
| | | | | | | |
| * | | | | | | eval: Refactor nr2char()ZyX2017-04-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds error messages, checks type and ignores the second argument. Currently utf_char2bytes is able to handle any 31-bit character, not limited by a unicode range. So checking for INT_MAX and not for something else: function yet uses `int`.
| * | | | | | | eval: Silence octal constant warningZyX2017-04-16
| | | | | | | |
| * | | | | | | eval: Fix useless NULL checkZyX2017-04-16
| | | | | | | | | | | | | | | | | | | | | | | | partial_name() as it is written now really cannot return NULL
| * | | | | | | eval: Refactor f_char2nrZyX2017-04-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With has_mbyte equal to 1 and &encoding always UTF-8 second argument is no longer useful: utf_ptr2char is the same as mb_ptr2char. Also changes function behaviour a bit: now if second argument is not a number it immediately returns with error, without bothering to get a character.
| * | | | | | | eval: Fix V547: `d == NULL` was already checked at line 2986ZyX2017-04-16
| | | | | | | |
| * | | | | | | eval: Silence eap->skip false positivesZyX2017-04-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | `lnum` starts at `eap->line2` in case of skipping, so cycle is always run at least once.
| * | | | | | | eval/typval_encode: Silence then/else equivalence warningZyX2017-04-16
| | | | | | | |
| * | | | | | | edit: Copy assert to before the warningZyX2017-04-16
| | | | | | | |
| * | | | | | | macros: Fix excessive checkZyX2017-04-16
| | | | | | | |
| * | | | | | | edit: Fix strange codeZyX2017-04-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Based on the flow it looks like ptr could not be NULL here: if ptr_arg is NULL ptr is compl_leader, if compl_leader is NULL function exits. This also applies to Vim as far as I see.
| * | | | | | | digraph: Fix errors due to has_mbyte and friends being fixedZyX2017-04-16
| | | | | | | |
| * | | | | | | digraph: Ignore false positiveZyX2017-04-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reversed order is intentional, digraphs allow swapping characters.
| * | | | | | | diff: Silence -V519ZyX2017-04-16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Not exactly a false positive, but previous assignment is a part of the pattern “change global, run code which uses it, change global back”.
| * | | | | | | *: Silence some false positivesZyX2017-04-16
| | | | | | | |
* | | | | | | | highlight: default Cursor to guibg=fg, guifg=bgMatthieu Coudron2017-04-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #6508
* | | | | | | | fold: foldMoveRange(): fix :move bug #6534Matthew Malcomson2017-04-17
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes #6540 In #6221 there was a mistake in calculating which folds need to be re-ordered. When there are no folds after those that have been adjusted, then `move_end` remains 0. This results in reverse_fold_order() swapping folds that have been adjusted with uninitialised folds in the remainder of the grow array. Add a check in foldMoveRange() to account for this case.
* | | | | | | Merge #6219 from jbradaric/vim-7.4.2170Justin M. Keyes2017-04-16
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | vim-patch:7.4.{2170,2180,2240,2241,2242}
| * | | | | | | eval.c: Code style fixesJurica Bradaric2017-04-15
| | | | | | | |
| * | | | | | | test_timers.vim: Adjust timing to handle difference in implementationJames McCoy2017-04-15
| | | | | | | |
| * | | | | | | vim-patch:7.4.2359James McCoy2017-04-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Memory leak in timer_start(). Solution: Check the right field to be NULL. https://github.com/vim/vim/commit/26fe0d56912e42c2b16a61b2480e19ba569aee98
| * | | | | | | vim-patch:7.4.2332James McCoy2017-04-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Crash when stop_timer() is called in a callback of a callback. Vim hangs when the timer callback uses too much time. Solution: Set tr_id to -1 when a timer is to be deleted. Don't keep calling callbacks forever. (Ozaki Kiichi) https://github.com/vim/vim/commit/75537a93e985ef32e6c267b06ce93629855dd983
| * | | | | | | vim-patch:7.4.2304James McCoy2017-04-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: In a timer callback the timer itself can't be found or stopped. (Thinca) Solution: Do not remove the timer from the list, remember whether it was freed. https://github.com/vim/vim/commit/417ccd7138d4d230d328de8b0d3892dd82ff1bee
| * | | | | | | vim-patch:7.4.2281James McCoy2017-04-06
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Timer test fails sometimes. Solution: Reduce minimum time by 1 msec. https://github.com/vim/vim/commit/0426bae2abede764d0dd366a28663d1c6e6ab0fe
| * | | | | | | Merge remote-tracking branch 'origin/master' into vim-7.4.2170James McCoy2017-04-05
| |\ \ \ \ \ \ \
| * | | | | | | | vim-patch:7.4.2242Jurica Bradaric2017-03-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Timer test sometimes fails. Solution: Increase the maximum time for callback timer test. https://github.com/vim/vim/commit/17f1347b867cbcc0ce380bf9a2466b4c31896f04
| * | | | | | | | vim-patch:7.4.2241Jurica Bradaric2017-03-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Timer test sometimes fails. Solution: Increase the maximum time for repeating timer. https://github.com/vim/vim/commit/973365dcc40a41e6b72ece56f15cebfee69b1329
| * | | | | | | | vim-patch:7.4.2240Jurica Bradaric2017-03-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Problem: Tests using the sleep time can be flaky. Solution: Use reltime() if available. (Partly by Shane Harper) https://github.com/vim/vim/commit/f267f8bdf777073e392ada5b31d837c7b6090eb4