libosmocore[master]: utils/conv_gen.py: improve application flexibility

tnt gerrit-no-reply at lists.osmocom.org
Mon Jan 23 13:09:03 UTC 2017


Patch Set 4:

(1 comment)

https://gerrit.osmocom.org/#/c/1584/4/utils/conv_gen.py
File utils/conv_gen.py:

Line 26: import sys, os, math, getopt
Why use getopt and not argparse as max suggested ? It's way more "pythonic".


-- 
To view, visit https://gerrit.osmocom.org/1584
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I0b476b00234c17f78b41d695cf3bfd13edb64c28
Gerrit-PatchSet: 4
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Owner: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Max <msuraev at sysmocom.de>
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Vadim Yanitskiy <axilirator at gmail.com>
Gerrit-Reviewer: tnt <tnt at 246tNt.com>
Gerrit-HasComments: Yes


More information about the gerrit-log mailing list