Use per-thread error storage (thanks to @chkothe)

Author: tstenner

src/common.cpp

@@ -12,7 +12,7 @@
 #pragma comment(lib, "winmm.lib")
 #endif
 
-char last_error[512];
+thread_local char last_error[512] = {0};
 
 extern "C" {
 

src/common.h

@@ -31,7 +31,7 @@ extern "C" {
 	"Please do not compile this with a lslboost version older than 1.45 because the library would otherwise not be protocol-compatible with builds using other versions."
 #endif
 
-extern char last_error[512];
+extern thread_local char last_error[512];
 
 // the highest supported protocol version
 // * 100 is the original version, supported by library versions 1.00+

testing/test_ext_streaminfo.cpp

@@ -1,10 +1,27 @@
 #include "catch.hpp"
 #include <lsl_cpp.h>
+#include <thread>
 
 namespace {
 
 TEST_CASE("streaminfo", "[streaminfo][basic]") {
 	CHECK_THROWS(lsl::stream_info("", "emptyname"));
-	CHECK(std::string("The name of a stream must be non-empty.") == lsl_last_error());
+	REQUIRE(std::string("The name of a stream must be non-empty.") == lsl_last_error());
+}
+
+TEST_CASE("multithreaded lsl_last_error", "[threading][basic]") {
+	CHECK_THROWS(lsl::stream_info("", "emptyname"));
+	std::thread([](){
+		CHECK_THROWS(lsl::stream_info("hasname","type", -1));
+		REQUIRE(std::string("The channel_count of a stream must be nonnegative.") == lsl_last_error());
+	}).join();
+	REQUIRE(std::string("The name of a stream must be non-empty.") == lsl_last_error());
+}
+
+/// Ensure that an overly long error message won't overflow the buffer
+TEST_CASE("lsl_last_error size", "[basic]") {
+	std::string invalidquery(512, '\'');
+	CHECK_THROWS(lsl::resolve_stream(invalidquery, 1, 0.1));
+	REQUIRE(lsl_last_error()[512] == 0);
 }
 } // namespace