Skip to content
Snippets Groups Projects
Commit 2ead91d0 authored by Olivier BICHLER's avatar Olivier BICHLER
Browse files

Fix for delAttr

parent 0f9a06d1
No related branches found
No related tags found
2 merge requests!212Version 0.3.0,!161Add support for namespaced attributes in DynamicAttributes
Pipeline #50517 passed
......@@ -144,17 +144,15 @@ public:
void delAttr(const std::string& name) {
const auto dot = name.find('.');
if (dot == name.npos) {
AIDGE_ASSERT(isPascalCase(name), "Aidge standard requires PascalCase for C++ Attributes for \"{}\".", name);
mAttrs.erase(name);
mAttrs.erase((isSnakeCase(name)) ? snakeToPascal(name) : name);
#ifdef PYBIND
mAttrsPy.erase(pascalToSnake(name));
mAttrsPy.erase((isPascalCase(name)) ? pascalToSnake(name) : name);
#endif
}
else {
const auto ns = name.substr(0, dot);
AIDGE_ASSERT(isPascalCase(ns), "Aidge standard requires PascalCase for C++ Attributes namespace for \"{}\".", ns);
const auto nsName = name.substr(dot + 1);
future_std::any_cast<DynamicAttributes&>(mAttrs.at(ns)).delAttr(nsName);
future_std::any_cast<DynamicAttributes&>(mAttrs.at((isSnakeCase(ns)) ? snakeToPascal(ns) : ns)).delAttr(nsName);
}
}
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment