IfcSchema declaration_by_name to be safe

Andrej730 · Andrej730 · commit 2a2efefac30c · 2024-09-17T16:28:02.000+05:00
Couldn't find if we actually use it anywhere in the code but better to keep it safe, especially because it's exposed to Python and by accident user might provide some too bit integer instead of a string.
diff --git a/src/ifcparse/IfcSchema.h b/src/ifcparse/IfcSchema.h
@@ -496,7 +496,7 @@ class IFC_PARSE_API schema_definition {
     }
 
     const declaration* declaration_by_name(size_t name) const {
-        return declarations_[name];
+        return declarations_.at(name);
     }
 
     const std::vector<const declaration*>& declarations() const { return declarations_; }

Original file line number	Diff line number	Diff line change
`@@ -496,7 +496,7 @@ class IFC_PARSE_API schema_definition {`
`496`	`496`	`}`
`497`	`497`
`498`	`498`	`const declaration* declaration_by_name(size_t name) const {`
`499`		`- return declarations_[name];`
	`499`	`+ return declarations_.at(name);`
`500`	`500`	`}`
`501`	`501`
`502`	`502`	`const std::vector<const declaration*>& declarations() const { return declarations_; }`