aboutsummaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAge
* 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
|
* 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”.