[PATCH wayland 2/3] server: move include directives before extern "C" wrapper
Hardening
rdp.effort at gmail.com
Mon Apr 18 06:16:37 UTC 2016
Le 17/04/2016 13:29, Emil Velikov a écrit :
> Analogous to previous commit but for the server(-core) header.
>
> Signed-off-by: Emil Velikov <emil.l.velikov at gmail.com>
> ---
> src/wayland-server-core.h | 8 ++++----
> 1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/src/wayland-server-core.h b/src/wayland-server-core.h
> index e8e1e9c..fa7f394 100644
> --- a/src/wayland-server-core.h
> +++ b/src/wayland-server-core.h
> @@ -26,15 +26,15 @@
> #ifndef WAYLAND_SERVER_CORE_H
> #define WAYLAND_SERVER_CORE_H
>
> -#ifdef __cplusplus
> -extern "C" {
> -#endif
> -
> #include <sys/types.h>
> #include <stdint.h>
> #include "wayland-util.h"
> #include "wayland-version.h"
>
> +#ifdef __cplusplus
> +extern "C" {
> +#endif
> +
> enum {
> WL_EVENT_READABLE = 0x01,
> WL_EVENT_WRITABLE = 0x02,
>
Shouldn't the extern declaration be moved to surround only the functions
declarations (and so not including type defines) ?
That's just a question, if a C++ specialist have an answer...
--
David FORT
website: http://www.hardening-consulting.com/
More information about the wayland-devel
mailing list