Looks good to me, but I have not run it. ;)
A couple of questions/comments:
1) I presume you have verified that all uses of "LOG" have been changed to
"logger"?
Given the scoping, It appears you should have gotten them all.
2) I would re-write the commit message thus:
Currently, ryu-manager emits traceback messages when killed via
KeyboardInterrupt. This patch traps the KeyboardInterrupt, and outputs an
appropriate log message instead.
Best,
Victor
--
Victor J. Orlikowski <> vjo@[cs.]duke.edu<http://duke.edu>
On Jul 21, 2016, at 2:59 AM, Iwase Yusuke
<iwase.yusu...@gmail.com<mailto:iwase.yusu...@gmail.com>> wrote:
Then, would you review my attached patch?
------------------------------------------------------------------------------
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.http://sdm.link/zohodev2dev
_______________________________________________
Ryu-devel mailing list
Ryu-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/ryu-devel