diff options
author | yonghong-song <ys114321@gmail.com> | 2019-01-09 19:43:23 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-01-09 19:43:23 -0800 |
commit | ae839790d6729cef9c26fef74a21555257f81dec (patch) | |
tree | 2848236115714a207fd234e2cc00e37d14320df0 | |
parent | 9b3b1274bd34870dd8b41f40ab85f394bbb01079 (diff) | |
download | bcc-ae839790d6729cef9c26fef74a21555257f81dec.tar.gz |
remove unused extern variables (#2125)
On fc29, compiling bcc from source, the following
compiler warnings are seen:
/home/yhs/work/bcc/src/cc/bcc_elf.c: In function ‘bcc_free_memory_with_file’:
/home/yhs/work/bcc/src/cc/bcc_elf.c:802:36: warning: unused variable ‘_fini’ [-Wunused-variable]
extern unsigned long _start, _fini;
^~~~~
/home/yhs/work/bcc/src/cc/bcc_elf.c:802:28: warning: unused variable ‘_start’ [-Wunused-variable]
extern unsigned long _start, _fini;
^~~~~~
These unused externs are accidentally introduced by
Commit 51480d0597cc ("implement free_bcc_memory() API").
This patch removed them.
Signed-off-by: Yonghong Song <yhs@fb.com>
-rw-r--r-- | src/cc/bcc_elf.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/cc/bcc_elf.c b/src/cc/bcc_elf.c index dfbbadec..c6745a2b 100644 --- a/src/cc/bcc_elf.c +++ b/src/cc/bcc_elf.c @@ -799,8 +799,6 @@ static int bcc_free_memory_with_file(const char *path) { saddr = (unsigned long)bcc_free_memory - sym_addr + header.sh_addr; eaddr = saddr + header.sh_size; - extern unsigned long _start, _fini; - // adjust saddr and eaddr, start addr needs to be page aligned saddr_n = (saddr + page_size - 1) & ~(page_size - 1); eaddr -= saddr_n - saddr; |