Updates the Python wrapper per Thomas Krijnens' suggestion regarding

Luke Brooks · Luke Brooks · commit 05442e33f241 · 2017-08-21T14:49:15.000+01:00
RAII.
diff --git a/src/ifcwrap/IfcParseWrapper.i b/src/ifcwrap/IfcParseWrapper.i
@@ -403,10 +403,10 @@ private:
 		return f;
 	}
     IfcParse::IfcFile* read(const std::string& data) {
+		char* copiedData = new char[data.length()];
+		memcpy(copiedData, data.c_str(), data.length());
 		IfcParse::IfcFile* f = new IfcParse::IfcFile();
-        std::string copiedData;
-        copiedData = data;
-		f->Init((void *)copiedData.c_str(), data.length());
+		f->Init((void *)copiedData, data.length());
 		return f;
 	}
 	const char* schema_identifier() {

Original file line number	Diff line number	Diff line change
`@@ -403,10 +403,10 @@ private:`
`403`	`403`	`return f;`
`404`	`404`	`}`
`405`	`405`	`IfcParse::IfcFile* read(const std::string& data) {`
	`406`	`+ char* copiedData = new char[data.length()];`
	`407`	`+ memcpy(copiedData, data.c_str(), data.length());`
`406`	`408`	`IfcParse::IfcFile* f = new IfcParse::IfcFile();`
`407`		`- std::string copiedData;`
`408`		`- copiedData = data;`
`409`		`- f->Init((void *)copiedData.c_str(), data.length());`
	`409`	`+ f->Init((void *)copiedData, data.length());`
`410`	`410`	`return f;`
`411`	`411`	`}`
`412`	`412`	`const char* schema_identifier() {`