Stefan Hajnoczi writes:
>> +    kwargs = {}
>> +
>> +    try:
>> +        tracetool.generate(sys.stdin, arg_format, arg_backend, **kwargs)

> If forgot to ask what kwargs is doing here?   Can we default to {}?
> Otherwise let's drop it for now since there are no options defined yet.

It's used later in the patch adding dtrace support. I just thought it would be
better to maintain the interface of 'tracetool.generate' instead of adding new
parameters later when necessary.


Lluis

-- 
 "And it's much the same thing with knowledge, for whenever you learn
 something new, the whole world becomes that much richer."
 -- The Princess of Pure Reason, as told by Norton Juster in The Phantom
 Tollbooth

Reply via email to