Attention is currently required from: pespin.
3 comments:
Patchset:
I see these 2 ORTP references, but they are probably wrong and can be dropped since they should come […]
I will make a follow-up patch removing these bogons. And yes, they are bogons because osmo-bts configure has no pkg-config calls for ortp, only libosmo-abis configure does.
File contrib/ber/Makefile.am:
Patch Set #2, Line 1: if ENABLE_ORTP
See how it's done for instance in osmo-bsc/src/Makefile. […]
Will do.
File src/common/bts.c:
you don't really need {} afaict, we are enforcing -std=gnu11 during build in configure. […]
But I prefer to have these braces here, as they keep the local var contained in this block, not accessible outside of it. Initially I just put `#ifdef HAVE_ORTP` around the two original executing-code lines, but then the result was a compiler warning about unused automatic variable when building with `--disable-ortp`. If we remove the braces while keeping the local var declaration inside the `#ifdef`, then subsequent code in this function either sees or not sees this local var depending on ortp build config - not good IMO.
To view, visit change 42197. To unsubscribe, or for help writing mail filters, visit settings.