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