On Fri, 10 Jun 2016 17:56:29 +0900 (JST)
FUJITA Tomonori <[email protected]> wrote:

> On Thu, 09 Jun 2016 19:30:37 +0900
> fumihiko kakuma <[email protected]> wrote:
> 
> > Thank you for review.
> > 
> > On Thu, 09 Jun 2016 17:25:21 +0900
> > IWAMOTO Toshihiro <[email protected]> wrote:
> > 
> >> At Thu,  9 Jun 2016 15:19:30 +0900,
> >> fumihiko kakuma wrote:
> >> > 
> >> > Signed-off-by: Fumihiko Kakuma <[email protected]>
> >> 
> >> View objects work fine for for statements.
> >> Do we need this patch?
> > 
> > Yes, however there is another itervalues() as a method which
> > return a iterator.
> > So I thought that values() method is expected to return list.
> 
> both methods are actually used?

itervalues() is not used and values() is used in some modules.
But it does not seem to have any problem in the case retuning view.
Should we use only values() method that return view ?

> 
> ------------------------------------------------------------------------------
> What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
> patterns at an interface-level. Reveals which users, apps, and protocols are 
> consuming the most bandwidth. Provides multi-vendor support for NetFlow, 
> J-Flow, sFlow and other flows. Make informed decisions using capacity 
> planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e
> _______________________________________________
> Ryu-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/ryu-devel

-- 
fumihiko kakuma <[email protected]>



------------------------------------------------------------------------------
What NetFlow Analyzer can do for you? Monitors network bandwidth and traffic
patterns at an interface-level. Reveals which users, apps, and protocols are 
consuming the most bandwidth. Provides multi-vendor support for NetFlow, 
J-Flow, sFlow and other flows. Make informed decisions using capacity 
planning reports. https://ad.doubleclick.net/ddm/clk/305295220;132659582;e
_______________________________________________
Ryu-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/ryu-devel

Reply via email to