[#1384] Preserve BSK logging C-wrapper test catch on MSVC

Route the _bskError C-wrapper test through a volatile C++ function
pointer on MSVC.  MSVC builds use /EHc, which allows the compiler to
assume extern "C" calls do not throw; that caused the BasiliskError from
_bskError to bypass the local typed catch and escape to GoogleTest even
though the error message was correct.

Keep the normal test path unchanged for other compilers while preserving
coverage that _bskError treats preformatted text as a literal message.

Author: schaubh

src/architecture/utilities/tests/test_bskLogging.cpp

@@ -69,7 +69,14 @@ TEST(BSKLogging, cBskErrorTreatsMessageAsText)
     BSKLogger bskLogger;
 
     try {
+#ifdef _MSC_VER
+        // Keep MSVC from applying /EHc's extern "C" non-throwing assumption here.
+        using CBskErrorCaller = void (*)(BSKLogger&, const char*);
+        CBskErrorCaller volatile cBskErrorCaller = callCBskError;
+        cBskErrorCaller(bskLogger, "preformatted %s message");
+#else
         callCBskError(bskLogger, "preformatted %s message");
+#endif
     } catch (const BasiliskError& error) {
         EXPECT_STREQ("preformatted %s message", error.what());
         return;