On Fri, 20 Feb 2015 10:23:34 +0100 Tobias Jungel <[email protected]> wrote:
> On Fr, 2015-02-20 at 17:56 +0900, FUJITA Tomonori wrote: >> On Thu, 12 Feb 2015 20:01:45 +0100 >> Tobias Jungel <[email protected]> wrote: >> >> > In case one of the kwargs is set to None the ofproto parsers for >> > versions 1.2, 1.3 and 1.4 fail. This patch skips k,v pairs having set v >> > to None. >> >> Why does an user need to pass an arugment set to None? > > well, its actually not the need for that, rather the API allows it and > its not checked. I had written a controller and passed just the **dict > including all the values. Currently I use a dict comprehension to wipe > the None values. > > Having said that it might be okay to have it in the documentation only. > Personally I like an API, that checks the users input. If we start such checking, then there are too many things to do rather than just checking None. I don't see it's worth. If the current documentation is not clear on this issue, please send a patch. ------------------------------------------------------------------------------ Download BIRT iHub F-Type - The Free Enterprise-Grade BIRT Server from Actuate! Instantly Supercharge Your Business Reports and Dashboards with Interactivity, Sharing, Native Excel Exports, App Integration & more Get technology previously reserved for billion-dollar corporations, FREE http://pubads.g.doubleclick.net/gampad/clk?id=190641631&iu=/4140/ostg.clktrk _______________________________________________ Ryu-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/ryu-devel
