From 762a8ad0f3dc93ff3d938f41af324620531b2923 Mon Sep 17 00:00:00 2001 From: Eliseo Martínez Date: Mon, 12 May 2014 12:12:27 +0200 Subject: 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 and also 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 through "vim.h" --> "os_unix_defs.h". Solution: Correctly include "nvim/strings.h" in every file needing it. --- src/nvim/crypt.c | 1 + 1 file changed, 1 insertion(+) (limited to 'src/nvim/crypt.c') diff --git a/src/nvim/crypt.c b/src/nvim/crypt.c index e0f49f7e40..2eb699d84e 100644 --- a/src/nvim/crypt.c +++ b/src/nvim/crypt.c @@ -17,6 +17,7 @@ #include "nvim/ex_getln.h" #include "nvim/message.h" #include "nvim/option.h" +#include "nvim/strings.h" static void make_crc_tab(void); -- cgit