aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* doc/CONTRIBUTING.md: Recommend merge-based workflow.Nikolai Aleksandrovich Pavlov2017-04-19
| | | | References #6435
* health.vim: Set 'iskeyword' to that of ft=help.Justin M. Keyes2017-04-19
|
* doc: fix typo (#6504)Shlomi Fish2017-04-19
|
* Merge #6533 'Fix PVS-studio warnings'Justin M. Keyes2017-04-17
|\
| * buffer: Adjust where do_buffer call is locatedZyX2017-04-16
| | | | | | | | It is located there in Vim, but in dd7657c1605246e8f7ade35184069a09dc254e84 position was for some reason swapped.
| * *: Fix linter errorsZyX2017-04-16
| |
| * regexp_nfa: Remove octal constantZyX2017-04-16
| |
| * regexp_nfa: Remove another has_mbyte/…-checking codeZyX2017-04-16
| |
| * regexp: Silence octal constant warningZyX2017-04-16
| |
| * regexp: Remove another has_mbyte/…-checking stuffZyX2017-04-16
| |
| * regexp: Fix warning about octal constantZyX2017-04-16
| |
| * tag: Fix “initialized twice successively” false positiveZyX2017-04-16
| |
| * tag: Silence “buffer underflow” warningZyX2017-04-16
| |
| * screen: Remove another portion of has_mbyte/friends-checking stuffZyX2017-04-16
| |
| * screen: Remove unneeded conditionZyX2017-04-16
| | | | | | Already checked in outer if()
| * 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
| |