[PATCH 0/2] Bug 28954 - add Linux Kernel symbol namespace support
Giuliano Procida
gprocida@google.com
Mon Mar 14 18:13:10 GMT 2022
Hi Dodji.
These two patches aim to add kernel symbol namespace support. There
are two bits of this that could be debated (and easily changed).
A. choice of XML attribute name
I've chosen "ns", mostly because "namespace" implies C++ namespace
everywhere else.
B. choice of representation
There are several possible levels of distinction of presence:
1. Non-kernel symbols cannot be exported to a namespace.
2. Kernel symbols can be exported globally and not to namespace.
3. Kernel symbols can be exported to a named namespace.
And there is the option of treating the empty namespace name string
specially. I've chosen to represent namespace internally as
optional<string> and not distinguish between cases 1 and 2.
Regards.
Giuliano Procida (2):
optional: add operator== and operator!=
add Linux kernel symbol namespace support
include/abg-cxx-compat.h | 16 +++++++++++++
include/abg-ir.h | 9 ++++++++
src/abg-comp-filter.cc | 39 +++++++++++++++++++++++++++++++-
src/abg-ir.cc | 27 +++++++++++++++++++++-
src/abg-reader.cc | 7 ++++++
src/abg-reporter-priv.cc | 12 ++++++++++
src/abg-symtab-reader.cc | 49 ++++++++++++++++++++++++++++++++++++++++
src/abg-writer.cc | 4 ++++
8 files changed, 161 insertions(+), 2 deletions(-)
--
2.35.1.723.g4982287a31-goog
More information about the Libabigail
mailing list