Am Dienstag 24 April 2012, 23:19:40 schrieb Peter Huewe:
> In the current code we asign vsize=8 and then fall through to the
> default and asign vsize=1. -> probably the break is missing here,
> otherwise we can remove the case.
> 
> Signed-off-by: Peter Huewe <peterhu...@gmx.de>
> ---
>  pevent/parse-events.c |    1 +
>  1 files changed, 1 insertions(+), 0 deletions(-)
> 
> diff --git a/pevent/parse-events.c b/pevent/parse-events.c
> index e1dde50..861612e 100644
> --- a/pevent/parse-events.c
> +++ b/pevent/parse-events.c
> @@ -3441,6 +3441,7 @@ static struct print_arg *make_bprint_args(char *fmt,
> void *data, int size, struc break;
>                               case 2:
>                                       vsize = 8;
> +                                     break;
>                               default:
>                                       vsize = ls; /* ? */
>                                       break;

Any reason for not picking up these two patches, or did you miss them?

Thanks,
Peter

_______________________________________________
Power mailing list
Power@bughost.org
https://bughost.org/mailman/listinfo/power

Reply via email to