osmo-bsc[master]: handover_test: explicitly wrap abis_rsl_sendmsg()

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Neels Hofmeyr gerrit-no-reply at lists.osmocom.org
Thu Mar 8 04:53:44 UTC 2018


Patch Set 1:

> It works as both the function caller and the function itself are in the same .c/.o file.

In this case the function caller is somewhere deep in libbsc, and it is calling a function originally defined somewhere deep in libosmo-abis. We are putting another definition that function in the local file, in the hope that the libbsc caller is diverted to the local function instead of libosmo-abis. Feels lucky to me, in the sense that the symbol is only pulled from the libosmo-abis library if it doesn't exist in the current ... erm ... "scope". Does it still work if libosmo-abis and libbsc both appear before the main.o on the linking cmdline? I'm not familiar with that.

Even if that magic works and it's only me not knowing about it, I still think it adds clarity if we *explicitly* wrap it?

BTW, so, if I by accident pick an already existing function name in my main.c, I silently override the other definition in the linking? Scary :) -- though I think I saw the compiler complaining about multiple definitions of the same function on occasions.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I05ce33a4e42bc82b9ce5060e2f811033f5a1f8d8
Gerrit-PatchSet: 1
Gerrit-Project: osmo-bsc
Gerrit-Branch: master
Gerrit-Owner: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-Reviewer: Harald Welte <laforge at gnumonks.org>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr <nhofmeyr at sysmocom.de>
Gerrit-HasComments: No



More information about the gerrit-log mailing list