Eliseo Martínez
4fe0a51844
Remove project int types: Case long_i: Replace with plain long.
...
Replace long_i with plain long.
long_i was just plain long, adding marker __w64, to be used by
Microsoft's compilers only, as an aid when transitioning from 32 bits to
64 bits. Purpose of this marker was, in fact, to make a 32 bit compiler
emit the same warnings that a 64 bit compiler would.
This __w64 marker is nowadays deprecated by said compilers, and use of a
real 64 bit compiler is recommended instead. See
http://msdn.microsoft.com/en-us/library/s04b5w00.aspx for details.
So, there's no reason to maintain this anymore, and thus is removed.
Refactoring long into int64_t is not attempted, as doing that in a bulk
way is too much complicated. That is left to be done later, on a
file-by-file basis, probably intermixed with file-by-file -Wconversion
activation.
Requested in #459 .
2014-05-03 10:04:49 -04:00
Eliseo Martínez
ad966753d4
Remove project int types: Case int_u: Replace with uint32_t.
...
- Replace int_u with uint32_t (same size, should give no problems).
In fact, only usage found seems to be dead code (some functions in
ui.h seem not to be used anymore).
- Remove int_u typedef.
Requested in #459 .
2014-05-03 10:04:49 -04:00
Eliseo Martínez
cb7fb8c008
Remove project int types: Case short_u: Replace with uint16_t.
...
- Replace short_u with uint16_t (same size, should give no problems).
- When possible, remove explicit downcasts so that they are found when
flag -Wconversion enabled.
- Remove short_u typedef.
Requested in #459 .
2014-05-03 10:04:49 -04:00
ZyX
37935130f9
Do not detect macros like VIM_TRUE as boolean values
2014-05-03 08:44:04 -04:00
Marco Hinz
8a729897f4
Improve test_eval.in
...
The old version required to change the range given to :w for each new
test. Now a new buffer is used instead.
2014-05-02 16:01:31 -03:00
John Schmidt
abecd0f394
Update README: progress section
2014-05-02 15:59:52 -03:00
Felipe Oliveira Carvalho
d1411aebd8
Rename buf_modname to modname as they've become the same
2014-05-02 15:58:33 -03:00
Felipe Oliveira Carvalho
9b56e3a4cc
Remove the always-FALSE shortname argument from buf_modname()
2014-05-02 15:58:33 -03:00
Felipe Oliveira Carvalho
89e07185e9
Remove the code related to the shortname option
2014-05-02 15:58:33 -03:00
Felipe Oliveira Carvalho
8500eeed86
Don't retry to make the tempname in write_viminfo() with shortname=1
...
ref #572
2014-05-02 15:58:33 -03:00
Chris Watkins
cb5acb705a
Remove checks for SHORT_FNAME and USE_LONG_FNAME.
...
They were only defined for MSDOS which is now unsupported.
2014-05-02 15:58:33 -03:00
Pavel Platto
17d97e725a
Remove two-iteration loop from check_keyword_id
2014-05-02 15:37:43 -03:00
Pavel Platto
97f02bb609
Remove two-iteration loop from expand_filename
2014-05-02 15:37:43 -03:00
Pavel Platto
1a946ad05f
Remove two-iteration loop from ExpandGeneric
2014-05-02 15:37:43 -03:00
John Schmidt
3f6fe2a888
Move concat_strings
from path.c
2014-05-02 15:36:13 -03:00
John Schmidt
7a4d24d4cc
Extract strings.c from misc2.c
2014-05-02 15:36:13 -03:00
Hinidu
018bb73296
Added missing #define guards in headers
2014-05-01 18:24:58 -04:00
Chris Watkins
67a157c08d
Replace 'alloc' with 'xmalloc' in some files.
...
Files changed: charset.c, buffer.c, diff.c, edit.c,
ex_cmds.c, ex_cmds2.c and ex_docmd.c.
The remaining alloc's in these files require more careful attention to
remove.
2014-05-01 16:59:11 -04:00
Justin M. Keyes
1b5217687a
revert #652
...
reverting broad cosmetic/style change because:
- increases merge-conflicts
- increases overhead of merging upstream Vim patches
- reasons for change are ambiguous, so default to no change
2014-05-01 14:00:31 -04:00
Joel Teichroeb
3b77a62a77
Remove incorrect trailing backslash
...
If this backslash is here and USE_VALGRIND is defined, valgrind complains that "else" is not a command.
2014-04-30 19:01:27 -07:00
Marco Hinz
5d65e7f279
vim-patch:7.4.272
...
Problem: Using just "$" does not cause an error message.
Solution: Check for empty environment variable name.
(Christian Brabandt)
https://code.google.com/p/vim/source/detail?r=00228400629e28384f7f52556c3c119ba0d0a44d
2014-04-30 11:21:46 -04:00
John Szakmeister
c79d27ee07
build: remove unnecessary flag from CMAKE_FLAGS
...
We now append the deps install area into CMAKE_PREFIX_PATH.
2014-04-30 05:41:14 -04:00
Marco Hinz
2e4613aecc
Remove NUL macro
2014-04-29 17:51:09 -03:00
Marco Hinz
046debb935
vim-patch:7.4.268
...
Problem: Using exists() on a funcref for a script-local function
does not work.
Solution: Translate <SNR> to the special byte sequence.
Add a test.
https://code.google.com/p/vim/source/detail?r=1a5ed2626b26a982e307a206572121a557adf709
2014-04-29 15:14:25 -03:00
Marco Hinz
f30a83f101
Refactor test_eval
2014-04-29 15:13:57 -03:00
Marco Hinz
48fa2be92b
vim-patch:7.4.269
...
Problem: CTRL-U in Insert mode does not work after using a cursor key.
(Pine Wu)
Solution: Use the original insert start position. (Christian Brabandt)
https://code.google.com/p/vim/source/detail?r=81c26975e8f9dc7435353581346542409403f296
2014-04-29 15:11:57 -03:00
Marco Hinz
35ca066039
vim-patch:7.4.266
...
Problem: Test 62 fails.
Solution: Set the language to C. (Christian Brabandt)
https://code.google.com/p/vim/source/detail?r=8f84e906d454a95d3167678a745dde9de442b604
2014-04-29 15:05:47 -03:00
Klemen Košir
3be437ba27
Changed variable types to fixed-size.
2014-04-29 15:05:05 -03:00
Klemen Košir
d87613fbb0
Added crypt.h doxygen comments.
2014-04-29 15:05:05 -03:00
Klemen Košir
8d14dd93b8
Cleanup crypt.h/.c to follow the style guide.
2014-04-29 15:05:05 -03:00
oni-link
05d29fb0b8
Cleanup
...
* Remove unnecessary else
* Move variable declaration nearer to point of use
* Add missing curly brackets
* Comment style
2014-04-29 15:04:21 -03:00
oni-link
bee4f4f724
Call to rettv_list_alloc cannot fail.
...
Clean up the use of rettv_list_alloc and remove error checks.
2014-04-29 15:04:21 -03:00
oni-link
dc9b680deb
Call to list_append_tv cannot fail.
...
Clean up the use of list_append_tv and remove error checks.
2014-04-29 15:04:21 -03:00
oni-link
6797a3e788
Call to list_append_number cannot fail.
...
Clean up the use of list_append_number and remove error checks.
2014-04-29 15:04:21 -03:00
Klemen Košir
1a5505c46b
Updated comments in spell.h/.c to follow the style guide.
2014-04-29 13:14:25 -03:00
Klemen Košir
463d0940e3
vim-patch:7.4.262
...
Problem: Duplicate code in regexec().
Solution: Add line_lbr flag to regexec_nl().
https://code.google.com/p/vim/source/detail?r=0ea551fa607dc443b97c2fba97dc0c9cb0bcf303
2014-04-29 10:32:04 -03:00
Nicolas Hillegeer
852f0aaae8
garray: comment and refactor a bit more
...
The following functions are affected:
- ga_remove_duplicate_strings
- ga_append
2014-04-29 09:29:10 -03:00
Nicolas Hillegeer
eea98c4d3b
garray: refactor and optimize ga_concat
...
I've checked all uses of ga_concat in the codebase:
- ex_cmds2.c
- ex_getln.c
- eval.c
- message.c
- regexp-nfa.c
- term.c
- spell.c
None of them concats the garray with itself. This makes it safe to use the
faster memcpy() instead of memmove(). This change was also documented.
2014-04-29 09:29:10 -03:00
Nicolas Hillegeer
767f0fae0b
garray: make parameter of ga_concat_strings const
2014-04-29 09:29:10 -03:00
Nicolas Hillegeer
b737b27e6a
garray: add unit tests for ga_concat_strings_sep
2014-04-29 09:29:10 -03:00
Nicolas Hillegeer
104000eff6
garray: implement ga_concat_strings_sep
...
A generalized version of ga_concat_strings that can handle any separator.
Reimplement ga_concat_strings on top of it.
2014-04-29 09:29:10 -03:00
Nicolas Hillegeer
cf68eda287
garray: reimplement ga_concat_strings with stpcpy
...
Less useless strlen(), stpcpy() is a more natural fit for this task.
ga_concat_strings() still has too much strlen() but that would be ugly to
remove for a function that's not used very often (just once in the current
codebase).
2014-04-29 09:29:10 -03:00
Nicolas Hillegeer
6b486f41d1
memory: implement xstp(n)cpy
...
Both are implemented to POSIX 2008 spec, though I'm not sure I'm happy about
stpncpy's behaviour.
2014-04-29 09:29:10 -03:00
Nicolas Hillegeer
466b73108f
garray: refactor ga_grow
...
- xrealloc will call xmalloc if the input pointer is NULL, no need to check
twice.
- use the early-quit idiom to decrease the indentation, which enhances
readability.
2014-04-29 09:29:10 -03:00
Nicolas Hillegeer
ce9c49f222
garray: add unit tests
...
Only append_ga_line() was not tested because it relies on global vim state.
2014-04-29 09:29:10 -03:00
John
42efbfd2fd
Test and refactor shorten_fname
and shorten_fname1
...
Rename `shorten_fname` -> `path_shorten_fname`
Rename `shorten_fname1` -> `path_shorten_fname_if_possible`
2014-04-29 09:24:34 -03:00
John Schmidt
c57002a7c3
Remove shorten_filenames
(dead code)
2014-04-29 09:24:34 -03:00
Justin M. Keyes
9a2c92e92c
Merge pull request #620 from watk/remove-dead-feats
...
[RDY] Remove references to FEAT_{X11,CLIPBOARD} and USE_XSMP.
2014-04-28 20:31:56 -04:00
Chris Watkins
c9fba7ca7c
Remove references to FEAT_{X11,CLIPBOARD} and USE_XSMP.
2014-04-28 16:46:34 -07:00
Nicolas Hillegeer
9e04e81ac7
test: no longer define ad-hoc C headers
...
With the improved cimport helper, we can just load what we want.
2014-04-28 16:17:25 -03:00