Use function pointer when choosing qa output function
Replace the chain of if/else statements deciding which output formatting
function to use with a function pointer wrt comments by vapier
diff --git a/bin/repoman b/bin/repoman
index c7a1c4c..888892b 100755
--- a/bin/repoman
+++ b/bin/repoman
@@ -2432,10 +2432,14 @@
f = formatter.AbstractFormatter(console_writer)
-if options.output_style == 'column':
- utilities.format_qa_output_column(f, stats, fails, dofull, dofail, options, qawarnings)
-else:
- utilities.format_qa_output(f, stats, fails, dofull, dofail, options, qawarnings)
+format_outputs = {
+ 'column': utilities.format_qa_output_column,
+ 'default': utilities.format_qa_output
+}
+
+format_output = format_outputs.get(options.output_style,
+ format_outputs['default'])
+format_output(f, stats, fails, dofull, dofail, options, qawarnings)
style_file.flush()
del console_writer, f, style_file