* Markus Armbruster ([email protected]) wrote: > "Dr. David Alan Gilbert" <[email protected]> writes: > > > * Markus Armbruster ([email protected]) wrote: > >> Change the qemu_strtosz() & friends to return -EINVAL when @endptr is > >> null and the conversion doesn't consume the string completely. > >> Matches how qemu_strtol() & friends work. > >> > >> Only test_qemu_strtosz_simple() passes a null @endptr. No functional > >> change there, because its conversion consumes the string. > >> > >> Simplify callers that use @endptr only to fail when it doesn't point > >> to '\0' to pass a null @endptr instead. > >> > >> Cc: Dr. David Alan Gilbert <[email protected]> > >> Cc: Eduardo Habkost <[email protected]> (maintainer:X86) > >> Cc: Kevin Wolf <[email protected]> (supporter:Block layer core) > >> Cc: Max Reitz <[email protected]> (supporter:Block layer core) > >> Cc: [email protected] (open list:Block layer core) > >> Signed-off-by: Markus Armbruster <[email protected]> > > > > Reviewed-by: Dr. David Alan Gilbert <[email protected]> > > > > (end and endptr are horribly confusing names in do_strtosz) > > Could rename for consistency with qemu_strtol() & friends: > > * Parameter @end to @endptr > > * Local variable @endptr to @ep > > Would that be useful?
Oh I wouldn't worry too much about it; and those perhaps aren't that much clearer. Dave -- Dr. David Alan Gilbert / [email protected] / Manchester, UK
