Re: [PATCH v3 0/7] MR8690: widl: Various metadata fixes

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about tools/widl/parser.y:
> ;
>
> contract_ver:
> -
>  aNUM					{ $$ = MAKEVERSION(0, $1.value); }
> - | aNUM '.' aNUM				{ $$ = MAKEVERSION($3.value, $1.value); }
> -
>  aNUM					{ $$ = make_version( 0, $1.value ); }
> - | aNUM '.' aNUM				{ $$ = make_version( $3.value, $1.value ); }
> ;
>
> contract_req
>          : decl_spec ',' contract_ver            {
> -
>                                              struct integer integer = {.value = $3};
> -
>                                              struct integer integer = {.value = $3->major | $3->minor };

```suggestion:-0+0
                                                  struct integer integer = {.value = ($3->major << 16) | $3->minor };
```

I think?