Attention is currently required from: falconia, pespin, fixeria.

View Change

1 comment:

• File include/osmocom/codec/codec.h:

  • Patch Set #1, Line 85: etsi_sid_class

    > @pespin - then how come none of the protocol constant definitions in <osmocom/gsm/protocol/gsm_XX_ […]

    pespin is correct in that they predate our naming conventions. In fact, they likely even predate libosmocore. Originally in 2008 we had all of the code in openbsc.git, and then we simply copied it to libosmocore.git when we needed them on the OsmocomBB side, without thinking much about it.

    meanwhile we do have naming rules (which should also be documented in the wiki). For a few #defines in some gsm/protocol/foo.h file I don't care too much about it (almost no source file will include them), but in terms of symbol naming of public API functions polluting the global namespace it is a concern. The lack of a proper hierarchical namespace is one of the sad historic parts of C...

To view, visit change 32183. To unsubscribe, or for help writing mail filters, visit settings.