Re: [libvirt] [PATCH 3/5] remote generator: Unify unsigned int notation in .x files

On 05/07/2011 06:28 AM, Matthias Bolte wrote: > Rename u_int and unsigned to unsigned int. This gets rid of some special > case code in the generator. > --- >  daemon/remote_generator.pl   |   10 +++--- >  src/remote/remote_protocol.x |   72 +++++++++++++++++++++--------------------- >  2 files changed, 41 insertions(+), 41 deletions(-) > > diff --git a/daemon/remote_generator.pl b/daemon/remote_generator.pl > index 8d62c37..e6a0e2c 100755 > --- a/daemon/remote_generator.pl > +++ b/daemon/remote_generator.pl > @@ -878,13 +878,13 @@ elsif ($opt_k) { >                      push(@setters_list, "args.$1.${1}_val = (char *)$1;"); >                      push(@setters_list, "args.$1.${1}_len = ${1}len;"); >                      push(@args_check_list, { name => "\"$1\"", arg => "${1}len", limit => $2 }); > -                } elsif ($args_member =~ m/^(.*) (\S+);/) { > -                    my $type_name = $1; > -                    my $arg_name = $2; > +                } elsif ($args_member =~ m/^(unsigned )?(int|hyper) (\S+);/) { > +                    my $type_name; > +                    my $arg_name = $3; > > +                    $type_name = $1 if ($1); > +                    $type_name .= $2; >                      $type_name =~ s/hyper/long/; > -                    $type_name =~ s/^unsigned$/unsigned int/; > -                    $type_name =~ s/u_int/unsigned int/; Makes sense.  Certainly makes life stricter for $type_name, which is a good thing for maintenance. > +++ b/src/remote/remote_protocol.x > @@ -541,8 +541,8 @@ struct remote_domain_interface_stats_ret { > >  struct remote_domain_memory_stats_args { >      remote_nonnull_domain dom; > -    u_int maxStats; > -    u_int flags; > +    unsigned int maxStats; > +    unsigned int flags; Good change; http://www.ietf.org/rfc/rfc4506.txt only documents 'unsigned int identifier' [4.2], and not 'u_int' nor 'unsigned' for declaring unsigned 32-bit values.  rpcgen may have accepted those other aliases as an extension (or via an intermediate typedef), but we might as well stick as close to the specs as possible, especially if it makes life easier for our generator. ACK.