[Libvir] PATCH: Add xstrtol variants for long long

Daniel P. Berrange berrange at redhat.com
Sat Jan 19 19:33:16 UTC 2008


On Sat, Jan 19, 2008 at 08:28:38PM +0100, Jim Meyering wrote:
> "Daniel P. Berrange" <berrange at redhat.com> wrote:
> 
> > Jim's xstrtol convenience function simplified the use of strtol, but only
> > support the 'int' variant. For the storage drivers I need a similar function
> > that will always be at minimum 64-bit since we may have files > 2 GB even
> > on 32-bit. So this adds a variant of the xstrtol functions  which use a
> > long long / unsigned long long type
> >
> > Regards,
> > Dan.
> >
> > diff -r 83e80c558f4d src/internal.h
> > --- a/src/internal.h	Wed Jan 16 09:28:01 2008 -0500
> > +++ b/src/internal.h	Wed Jan 16 09:28:05 2008 -0500
> > @@ -304,6 +304,42 @@ xstrtol_ui(char const *s, char **end_ptr
> >      return 0;
> >  }
> >
> > +static inline int
> > +xstrtol_ll(char const *s, char **end_ptr, int base, long long *result)
> 
> ACK.
> 
> FYI, no harm of course, but I don't see a
> use of xstrtol_ll (yet?) in the queue.

Yes, I only use the xstrtol_ull  variant in the drivers, but figured we 
might as well define the whole set for completeness.

Dan.
-- 
|=- Red Hat, Engineering, Emerging Technologies, Boston.  +1 978 392 2496 -=|
|=-           Perl modules: http://search.cpan.org/~danberr/              -=|
|=-               Projects: http://freshmeat.net/~danielpb/               -=|
|=-  GnuPG: 7D3B9505   F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505  -=| 




More information about the libvir-list mailing list