On 10/22/10 09:59, Markus Armbruster wrote: > jes.soren...@redhat.com writes: >> From: Jes Sorensen <jes.soren...@redhat.com> >> This patch introduces cutils.c: strtosz() and gets rid of the >> multiple custom hacks for parsing byte sizes. In addition it adds >> supports for specifying human style sizes such as 1.5G. Last it >> eliminates the horrible abuse of a float to store the byte size for >> migrate_set_speed in the monitor. >> >> Note, this is tested on Linux and build tested for win32 using >> mingw32. >> >> v9: I worked through a couple of revisions directly with Markus and I >> think I got it right finally. > > I'd prefer to have strtosz() to match strtol() & friends and not > restrict suffixes. But this code does what it claims to do, as far as I > can see, so: > > ACK series
Thanks! I thought about this a fair bit and I believe doing the full test in the function is the most valuable. It's a personal preference obviously. I think it's a win to do it here since it simplifies the caller code. Would be great to get this applied so I can get it off my plate :) Cheers, Jes