[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