libosmocore[master]: abis_nm: Add abis_nm_admin_name() API
gerrit-no-reply at lists.osmocom.org
Mon Oct 23 01:22:06 UTC 2017
Patch Set 1:
For an example, see rsl_act_type_name() in libosmocore/include/osmocom/gsm/rsl.h
> So now
> there's the conflict in whether we align with current best
> practises or with the style that's used in the file so far.
Adding a static inline function to the header for this function alone is what I'd do. (And also publish the _names if it's not published yet.) And leave all the rest the way it is, for hysterical raisins.
> Also, this is yet another change that will break compilation with
> older libosmocore version. For I believe *very* questionable gain.
What change will break anything? Aligning the rest of the file to static inline functions? Then let's not?
To view, visit https://gerrit.osmocom.org/4355
To unsubscribe, visit https://gerrit.osmocom.org/settings
Gerrit-Owner: Pau Espin Pedrol <pespin at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
More information about the gerrit-log