aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* screen: Silence NULL dereference false positiveZyX2017-04-16
| | | | Based on the loop condition when shl_flag is true cur != NULL.
* screen: Silence “buffer underflow” warningZyX2017-04-16
|
* screen: Remove unneeded checkZyX2017-04-16
|
* quicfix: Avoid possible NULL dereferenceZyX2017-04-16
|
* quicfix: Remove duplicate conditionZyX2017-04-16
|
* 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
|
* 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
| |\