sal: log windows traces to debugger console, take two

This reverts most of commit 4c5b4752
and just keeps the OutputDebugStringA() call unconditionally.

Change-Id: Ia838052b3147ca2d66c7d28b04eadd9700244669
Reviewed-on: https://gerrit.libreoffice.org/54029Tested-by: Jenkins <ci@libreoffice.org>
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>

config_host/config_options.h.in

@@ -13,10 +13,4 @@
 
 #define ENABLE_RUNTIME_OPTIMIZATIONS 0
 
-/*
- * If this is a production build (where we might want to cut down on
- * debugging options)
- */
-#define ENABLE_RELEASE_BUILD 0
-
 #endif

configure.ac

@@ -2243,13 +2243,11 @@ dnl ===================================================================
 dnl Test whether build target is Release Build
 dnl ===================================================================
 AC_MSG_CHECKING([whether build target is Release Build])
-ENABLE_RELEASE_BUILD=
-
 if test "$enable_release_build" = "" -o "$enable_release_build" = "no"; then
     AC_MSG_RESULT([no])
+    ENABLE_RELEASE_BUILD=
 else
     AC_MSG_RESULT([yes])
-    AC_DEFINE(ENABLE_RELEASE_BUILD)
     ENABLE_RELEASE_BUILD=TRUE
 fi
 AC_SUBST(ENABLE_RELEASE_BUILD)

sal/osl/all/log.cxx

@@ -22,7 +22,6 @@
 #include <fstream>
 
 #include <config_global.h>
-#include <config_options.h>
 #include <osl/thread.hxx>
 #include <rtl/string.h>
 #include <sal/detail/log.h>
@@ -331,12 +330,9 @@ void sal_detail_log(
             // write to Windows debugger console, too
             OutputDebugStringA(s.str().c_str());
 #endif
-#if ! (defined(WNT) && ENABLE_RELEASE_BUILD)
-            // on Windows deployments, no one reads console output
             s << '\n';
             std::fputs(s.str().c_str(), stderr);
             std::fflush(stderr);
-#endif
         }
     }
 #endif