laforge has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-ggsn/+/17758 )

Change subject: sgsnemu: Refactor arg/cfg parser to use talloc
......................................................................


Patch Set 2:

Are you saying gengetopt on the .ggo file is not working anymore today?

If it ist still working, I don't think we should break it.  A memory leak in 
the command line argument parsing is not really a serious problem.  We're 
talking about small amounts of memory leaked once at startup only.


--
To view, visit https://gerrit.osmocom.org/c/osmo-ggsn/+/17758
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ggsn
Gerrit-Branch: master
Gerrit-Change-Id: Ie0d48a12a8349be5a859a03191421cbd9cc324cc
Gerrit-Change-Number: 17758
Gerrit-PatchSet: 2
Gerrit-Owner: pespin <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-Reviewer: pespin <[email protected]>
Gerrit-Comment-Date: Tue, 14 Apr 2020 13:10:11 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment

Reply via email to