[lvm-devel] [PATCH] (5/11) vg_read error returns, part 1

[Dave Wysochanski]

On Thu, 2008-10-30 at 19:04 +0100, Petr Rockai wrote:
> diff -rN -p -u old-lvmlib-b/lib/metadata/metadata.c
> new-lvmlib-b/lib/metadata/metadata.c
> --- old-lvmlib-b/lib/metadata/metadata.c        2008-10-30
> 18:17:02.505837415 +0100
> +++ new-lvmlib-b/lib/metadata/metadata.c        2008-10-30
> 18:17:02.577837348 +0100
> @@ -2371,42 +2371,66 @@ int pv_analyze(struct cmd_context *cmd, 
>   * 0 - fail
>   * 1 - success
>   */
> -int vg_check_status(const struct volume_group *vg, uint32_t status)
> +static int _vg_check_status(const struct volume_group *vg, uint32_t
> status)
>  {
> +       uint32_t ret = 0;
>         if ((status & CLUSTERED) &&
>             (vg_is_clustered(vg)) && !locking_is_clustered() &&
>             !lockingfailed()) {
>                 log_error("Skipping clustered volume group %s",
> vg->name);
> -               return 0;
> +               ret |= FAILED_CLUSTERED;
> 

Need uint32_t for function return value consistency with internal
variable.  Probably want to move/fix the comment as well.