[ovs-dev] [PATCH 2/3] ovs-vsctl: Introduce new single-line output format.

Ben Pfaff blp at nicira.com
Fri Oct 16 16:37:49 UTC 2009


<reid at nicira.com> writes:

> Looks good.  
>> +    return ["%s" % vlan]
>> +                print '\\n'.join([s.replace('\\', '\\\\') for s in
> output])
> If you think you might ever want the return values of functions in their
> non-string form, you could save some worry and use
> str(s).replace(..)  instead of  s.replace(..)

I like your approach better anyhow.  Fixed.




More information about the dev mailing list