On Friday, November 02, 2012 11:22:04 AM David Malcolm wrote: > On Thu, 2012-11-01 at 11:20 -0400, Paul Moore wrote: > > ---------- Forwarded Message ---------- > > > > Subject: [RFC PATCH v2 2/3] tests: add basic support for python tests > > Date: Thursday, November 01, 2012, 11:18:22 AM > > From: Paul Moore <[email protected]> > > To: [email protected] > > CC: [email protected], [email protected] > > > > Signed-off-by: Paul Moore <[email protected]> > > --- > > > > tests/regression | 42 ++++++++++++++++++++++++++++++----------- > > tests/util.c | 4 ++-- > > tests/util.py | 56 > > > > ++++++++++++++++++++++++++++++++++++++++++++++++++++++ > > > > 3 files changed, 89 insertions(+), 13 deletions(-) > > create mode 100644 tests/util.py > > [...] > > > diff --git a/tests/util.py b/tests/util.py > > [...] > > > + > > > +def UtilGetOpt(): > Minor naming nit: don't use CamelCase for functions (save that for class > names), and "util" is already implied, given that Python has > namespacing. > > So maybe just call these > > def get_opt(): > > and: > > def filter_output(args, ctx):
Okay, easy enough. -- paul moore security and virtualization @ redhat ------------------------------------------------------------------------------ LogMeIn Central: Instant, anywhere, Remote PC access and management. Stay in control, update software, and manage PCs from one command center Diagnose problems and improve visibility into emerging IT issues Automate, monitor and manage. Do more in less time with Central http://p.sf.net/sfu/logmein12331_d2d _______________________________________________ libseccomp-discuss mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/libseccomp-discuss
