good for merge

On Mon, 2010-05-17 at 12:02 +1000, Angus Salkeld wrote:
> in the body it can return -1, and callers check for -1.
> but the return type is unsigned int?
> 
> Signed-off-by: Angus Salkeld <[email protected]>
> ---
>  exec/logsys.c                    |    2 +-
>  include/corosync/engine/logsys.h |    2 +-
>  2 files changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/exec/logsys.c b/exec/logsys.c
> index dc4e1d6..5dabe75 100644
> --- a/exec/logsys.c
> +++ b/exec/logsys.c
> @@ -1023,7 +1023,7 @@ int _logsys_system_setup(
>       return (0);
>  }
>  
> -unsigned int _logsys_subsys_create (const char *subsys)
> +int _logsys_subsys_create (const char *subsys)
>  {
>       int i;
>  
> diff --git a/include/corosync/engine/logsys.h 
> b/include/corosync/engine/logsys.h
> index 19e5dae..419dbc7 100644
> --- a/include/corosync/engine/logsys.h
> +++ b/include/corosync/engine/logsys.h
> @@ -167,7 +167,7 @@ extern int _logsys_system_setup(
>  extern int _logsys_config_subsys_get (
>       const char *subsys);
>  
> -extern unsigned int _logsys_subsys_create (const char *subsys);
> +extern int _logsys_subsys_create (const char *subsys);
>  
>  extern int _logsys_rec_init (unsigned int size);
>  

_______________________________________________
Openais mailing list
[email protected]
https://lists.linux-foundation.org/mailman/listinfo/openais

Reply via email to