Commit Graph

2352 Commits

Author SHA1 Message Date
Justin M. Keyes
9aa6cb0546 Merge pull request #1624 from Pyrohh/doc-fixes
doc: Misc. improvements to nvim-related docs
2014-12-08 00:10:40 -05:00
Michael Reed
593d63f1d5 doc: Misc. improvements to nvim-related docs
- Improved wording in a few places for clarity
- Various capitalization/grammar fixes
- Change references to Neovim as 'editor' to 'Nvim'
- Be consistent regarding utilization of vim's documentation features,
  e.g. unnamedclip -> |unnamedclip|
- Reflowed all changed paragraphs accordingly

- Add spaces before parentheses
- Remove trailing whitespace
- Standardize single spaces after periods. Vim's docs use two for the
  most part, but Nvim's use one mainly, so just follow Nvim's conventions
2014-12-07 23:45:28 -05:00
Justin M. Keyes
59bf1678cb Merge pull request #1623 from Pyrohh/cleanup-shortname
Docs: Remove 'shortname' references
2014-12-07 20:05:41 -05:00
Michael Reed
1cb7349302 runtime: Cleanup refs to removed options
- {no,inv,}shortname
- {no,inv,}sn
- cm, key (remnants of cryptmethod)
2014-12-07 13:51:05 -05:00
Michael Reed
8b35090c64 docs: Remove shortname references.
In addition, align the removal notice next to 'key' with other
such notices (options.txt).
2014-12-07 02:32:39 -05:00
Justin M. Keyes
fa9d44374b Merge pull request #1589 from splinterofchaos/fix-const-atter
strings: Remove NONNUL_ALL from NULL-taking functions.
2014-12-06 20:20:33 -05:00
Justin M. Keyes
01fc0efdca Merge pull request #1341 from splinterofchaos/api-nul
Api: Improve Nul handling
2014-12-06 20:19:08 -05:00
Scott Prager
cea5092f40 api: Add tests for when not to handle NULs. 2014-12-06 17:29:39 -05:00
Scott Prager
460843b4cd api: Handle NULs and newlines in buffer_*_line. 2014-12-06 17:29:38 -05:00
Scott Prager
4964d65362 strings: Remove NONNUL_ALL from NULL-taking functions.
vim_strup() is only used in a couple places, with no NULLs possible, so
keep the attribute and remove the NULL check.
2014-12-06 14:41:21 -05:00
Justin M. Keyes
dcee93f8ea Merge pull request #1618 from elmart/clang-analysis-fixes-5
Fix clang analysis warnings. (5)
2014-12-06 12:08:15 -05:00
Justin M. Keyes
85e9bab936 Merge pull request #1510 from three-comrades/testnocp
Allow all tests to pass with 'nocp'
2014-12-06 12:03:52 -05:00
Eliseo Martínez
734ac625bb Fix warnings: message.c: msg_attr_keep(): Np dereference: FP.
Problem    : Dereference of null pointer @ 179.
Diagnostic : False positive.
Rationale  : Error occurs if `s` paramater is null, which should not
             happen.
Resolution : Declare parameter as non null.
2014-12-06 17:40:12 +01:00
Eliseo Martínez
a71c5e9eb9 Fix warnings: edit.c: ins_compl_next_buf(): Np dereference: MI.
Problem    : Dereference of null pointer @ 3247.
Diagnostic : Multithreading issue.
Rationale  : Problem only occurs if global `ctrl_x_mode` is modified
             while calling function is executing.
Solution   : Use local copy instead of global.
2014-12-06 17:40:12 +01:00
Julian Mehne
42891ea9d9 Make migrated test33 more similar to old legacy test.
Remove `expandtab` and add a missing trailing space from the legacy test.
2014-12-06 13:43:14 +01:00
Julian Mehne
6cba2eef21 Allow the rest of the old tests to pass with nocp.
test40.in, test60.in, test_breakindent.in, test_listlbr.in, test_listlbr_utf8.in:
    To avoid <tab> triggering wildmode, set wildchar to 'cp' default.

test_breakindent.ok:
    Modify expectations to avoid cpoptions+=n
2014-12-06 13:43:14 +01:00
Joel Teichroeb
62bd20b726 Allow most old tests to pass with nocp.
test69.in:
    because 'nocp' sets 'esckeys', add an extra <esc> to avoid 'tm' pause.
2014-12-06 13:43:14 +01:00
Thiago de Arruda
b574cb6422 Merge PR #1620 'Fix 32bit unibilium build' 2014-12-06 09:34:51 -03:00
Thiago de Arruda
83d9f9b768 deps: Fix unibilium/libtickit URLs 2014-12-06 09:22:03 -03:00
Thiago de Arruda
f1a934808a Revert "deps: Fix 32-bit build of lib{unibilium,termkey,tickit}"
This reverts commit f76df24753.
2014-12-06 09:15:53 -03:00
Thiago de Arruda
33530ac9a4 Merge PR #1613 'Fix shell command output' 2014-12-06 08:08:47 -03:00
Thiago de Arruda
f76df24753 deps: Fix 32-bit build of lib{unibilium,termkey,tickit} 2014-12-06 07:54:59 -03:00
Thiago de Arruda
3b435621a5 shell: Fix shell command output
Shell command output was broken in @8a5a8db, which refactored nvim to no longer
switch to cooked mode(linefeeds are processed differently).

Fix the problem by refactoring write_output to accept to extra arguments that
control the flushing behavior and where data will be written to: buffer or
directly to the screen.
2014-12-06 07:50:03 -03:00
Thiago de Arruda
40adddd8e5 Merge PR #1616 'Update lua client' 2014-12-05 15:00:09 -03:00
Thiago de Arruda
a31b06c44d deps: Update lua client 2014-12-05 14:23:37 -03:00
Justin M. Keyes
45ae6eadfd Merge pull request #1609 from jszakmeister/use-plain-terminal-for-tests
build: use plainTerminal for tests on Travis CI
2014-12-04 10:32:12 -05:00
John Szakmeister
82748c36ef build: use plainTerminal for tests on Travis CI 2014-12-04 05:03:48 -05:00
Justin M. Keyes
81530e5816 Merge pull request #1563 from Pyrohh/cleanup-docs-encryption
doc: remove remaining crypto references
2014-12-03 17:33:33 -05:00
Michael Reed
497a688ebc Remove remaining crypto references
Closes https://github.com/neovim/docs/pull/26

Also added stubs for 'cryptmethod' and 'key', and placeholders for
explanation regarding removal of crypto functionality.
2014-12-03 14:46:37 -05:00
Justin M. Keyes
930e58c56d Merge pull request #1534 from oni-link/fix.leak.detected.in.1510
Fix memory leak detected in #1510.
2014-12-03 09:44:08 -05:00
Thiago de Arruda
cb86eca91f Merge PR #1603 'Small refactoring and dependencies update' 2014-12-03 10:35:25 -03:00
Thiago de Arruda
cc34c90df7 deps: Update unibilium and lua client 2014-12-03 10:12:41 -03:00
Thiago de Arruda
8b6473bd41 shell: Remove kShellOptCooked from ShellOpts 2014-12-03 10:12:12 -03:00
Thiago de Arruda
bf3a94ee51 time: Inline microdelay into os_microdelay 2014-12-03 10:03:39 -03:00
oni-link
eae3105ee3 channel.c: Fix for heap-use-after-free
ASAN detected this heap-use-after-free.
A job started by channel_from_job() could terminate and result in a call
to free_channel(), while channel_send_call() was still active/pending
and accessing Channel elements.

Original patch by @tarruda.
2014-12-03 13:41:17 +01:00
oni-link
e2e63832e3 Fix memory leak detected in PR 1510.
LSAN/ASAN detected, on an error code path, that not all elements of a
struct ChannelCallFrame were freed.
2014-12-03 13:41:17 +01:00
John Szakmeister
41f1678767 Revert "Merge pull request #1587 from equalsraf/tb-env-configs"
This reverts commit 53bccaaf99, reversing
changes made to 4049002f6b.
2014-12-03 07:29:03 -05:00
John Szakmeister
53bccaaf99 Merge pull request #1587 from equalsraf/tb-env-configs
Don't use env vars for configuration time options
2014-12-03 06:53:09 -05:00
John Szakmeister
4049002f6b Merge pull request #877 from equalsraf/tb-gendeclarations-windows
Refactor declaration generation error detection
2014-12-03 06:47:01 -05:00
John Szakmeister
788a5cf478 Merge pull request #1542 from jszakmeister/upgrade-libuv
Update to libuv 1.0.1.
2014-12-03 06:19:47 -05:00
John Szakmeister
279c519e33 build: fix a typo in a comment of FindLuaJit.cmake 2014-12-02 18:44:15 -05:00
Justin M. Keyes
81394c9d73 Merge pull request #1596 from elmart/fix-newline-subst
Fix newline substitution: Adapt to upstream patch.
2014-12-02 15:15:37 -05:00
Eliseo Martínez
d146b7c7ca Fix newline substitution: Adapt to upstream patch.
Fix previous changes to be compatible with agreed changes to upstream
vim (https://code.google.com/p/vim/issues/detail?id=287).
2014-12-02 19:04:30 +01:00
Justin M. Keyes
26b2f9ab11 Merge pull request #1567 from bfredl/systemlist
systemlist: add `keepempty` option to preserve final newline
2014-12-02 11:41:14 -05:00
Rui Abreu Ferreira
22f30cfca0 When SANITIZE=ON headers need -DEXITFREE 2014-12-02 16:21:26 +00:00
Björn Linse
b3151af69c systemlist: test empty lines in beginning and middle of output 2014-12-02 16:53:16 +01:00
Björn Linse
1464b0eda2 systemlist: add keepempty option to preserve final newline 2014-12-02 16:50:52 +01:00
Rui Abreu Ferreira
823a7ae61a Only use SANITIZE for nvim target
- Clang's Address Sanitizer options may cause problems when running
  the unit tests, restrict the use of these options to the nvim target.
2014-12-02 14:36:47 +00:00
Rui Abreu Ferreira
6e9af3c5dc Don't use env vars for configuration time options
- As a general rule of thumb one shouldn't use environment variables
  for setting configuration options for CMake. The reason for this is
  we don't know when CMake will be executed and re-evaluate that variable.
- e.g. If we run cmake a first time with a var set, and then run make on
  a second session (with no var) and cmake is called because a
  dependency changed, the option would be disabled
- This commit removes the use of environment vars from
  src/nvim/CMakeLists.txt entirely
- Removed SKIP_UNITTEST since it could only be used to remove a target
  at configuration time (and the target was optional anyway)
- Turned SANITIZE into an option, clang-asan.sh now passes cmake
  -DSANITIZE=ON
- Removed SKIP_EXEC since it was disabling a target at configuration time
  (not being used)
2014-12-02 14:36:47 +00:00
Thiago de Arruda
ecf1e672e1 deps: Use unibilium URL with Makefile fixed for OS X
Close #1594
2014-12-02 09:52:55 -03:00