diff options
author | Eliseo Martínez <eliseomarmol@gmail.com> | 2014-05-12 12:12:27 +0200 |
---|---|---|
committer | Eliseo Martínez <eliseomarmol@gmail.com> | 2014-05-15 20:46:02 +0200 |
commit | 762a8ad0f3dc93ff3d938f41af324620531b2923 (patch) | |
tree | 7e95d5a7fa3e919972f70e069489c9cedce3f667 /src/nvim/diff.c | |
parent | 409cc138f2de121288de3ffb930354b94af5edcd (diff) | |
download | rneovim-762a8ad0f3dc93ff3d938f41af324620531b2923.tar.gz rneovim-762a8ad0f3dc93ff3d938f41af324620531b2923.tar.bz2 rneovim-762a8ad0f3dc93ff3d938f41af324620531b2923.zip |
Introduce nvim namespace: Fix unmasked strings.h issue.
Problem: Now that nvim/strings.h is correctly namespaced, an issue
that had been masked until now arises:
When compiling, we get a lot of errors because of everywhere
the functions in nvim/strings.h are used, there's no include
to import them.
But, how could this compile and work previously, then? It
turns out that:
- In every such case, we are also including vim.h, which in
turn includes os_unix_defs.h.
- os_unix_defs.h includes <string.h> and also <strings.h> in
some systems (e.g. OSX).
- Build had been modified previously to (even when importing
system headers), prefer equally-named local ones. That was
in fact done as a previous attempt to solve the same issue
we are trying to solve another way now.
So, we were including our "strings.h" as a side-effect of
including <strings.h> through "vim.h" --> "os_unix_defs.h".
Solution: Correctly include "nvim/strings.h" in every file needing it.
Diffstat (limited to 'src/nvim/diff.c')
-rw-r--r-- | src/nvim/diff.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/nvim/diff.c b/src/nvim/diff.c index a5b4b64751..0e5d1c20b7 100644 --- a/src/nvim/diff.c +++ b/src/nvim/diff.c @@ -23,6 +23,7 @@ #include "nvim/option.h" #include "nvim/path.h" #include "nvim/screen.h" +#include "nvim/strings.h" #include "nvim/undo.h" #include "nvim/window.h" #include "nvim/os/os.h" |