[5/6] eal/windows: fix the curly braces mismatch problem
Checks
Commit Message
Supplement the first half of braces for the extern "C" block
Fixes: 99a2dd955fba6 ("lib: remove librte_ prefix from directory names")
Cc: bruce.richardson@intel.com
Signed-off-by: Weiguo Li <liwg06@foxmail.com>
---
lib/eal/windows/include/dirent.h | 4 ++++
1 file changed, 4 insertions(+)
Comments
2022-02-06 08:44 (UTC+0800), Weiguo Li:
> Supplement the first half of braces for the extern "C" block
>
> Fixes: 99a2dd955fba6 ("lib: remove librte_ prefix from directory names")
> Cc: bruce.richardson@intel.com
>
> Signed-off-by: Weiguo Li <liwg06@foxmail.com>
> ---
> lib/eal/windows/include/dirent.h | 4 ++++
> 1 file changed, 4 insertions(+)
>
> diff --git a/lib/eal/windows/include/dirent.h b/lib/eal/windows/include/dirent.h
> index 34eb077f8c..0a399d3145 100644
> --- a/lib/eal/windows/include/dirent.h
> +++ b/lib/eal/windows/include/dirent.h
> @@ -28,6 +28,10 @@
> #include <sys/stat.h>
> #include <errno.h>
>
> +#ifdef __cplusplus
> +extern "C" {
> +#endif
> +
> /* Maximum length of file name */
> #if !defined(PATH_MAX)
> # define PATH_MAX MAX_PATH
This file is not public, used only from DPDK C code.
Rather then adding the opening part, the closing part should be removed.
The "Fixes" tag is incorrect, should be:
Fixes: 6e1ed4cbbe99 ("eal/windows: add dirent implementation")
Cc: pallavi.kadam@intel.com
On Sun, 6 Feb 2022 19:23:03 +0300, Dmitry Kozlyuk wrote:
> This file is not public, used only from DPDK C code.
> Rather then adding the opening part, the closing part should be removed.
> The "Fixes" tag is incorrect, should be:
> Fixes: 6e1ed4cbbe99 ("eal/windows: add dirent implementation")
> Cc: pallavi.kadam@intel.com
>
Hi Dmitry,
Thanks for your remind. I'll update the patch later.
-Weiguo
@@ -28,6 +28,10 @@
#include <sys/stat.h>
#include <errno.h>
+#ifdef __cplusplus
+extern "C" {
+#endif
+
/* Maximum length of file name */
#if !defined(PATH_MAX)
# define PATH_MAX MAX_PATH