Re: vmctl: parse_size(): Use local variable instead of function parameter

2019-12-16 Thread Pratik Vyas
* Klemens Nanni [2019-12-16 23:42:04 +0100]: On Fri, Dec 06, 2019 at 06:49:52PM +0100, Klemens Nanni wrote: The parse_size() wrapper around scan_scaled(3) writes its intermediate result to the function argument which is always passed as literal zero. This seems odd, the function parameter has

Re: vmctl: parse_size(): Use local variable instead of function parameter

2019-12-16 Thread Klemens Nanni
On Fri, Dec 06, 2019 at 06:49:52PM +0100, Klemens Nanni wrote: > The parse_size() wrapper around scan_scaled(3) writes its intermediate > result to the function argument which is always passed as literal zero. > > This seems odd, the function parameter has no meaning but merely serves > as storage

vmctl: parse_size(): Use local variable instead of function parameter

2019-12-06 Thread Klemens Nanni
The parse_size() wrapper around scan_scaled(3) writes its intermediate result to the function argument which is always passed as literal zero. This seems odd, the function parameter has no meaning but merely serves as storage, so let's use a proper function scoped variable instead. OK? Index: m