Fixed bug where STDLogSink would wrap to bogus values when redirecting stdout to anything but a pty

azonenberg · azonenberg · commit c7138bb0c54f · 2017-03-02T19:33:33.000-08:00
diff --git a/STDLogSink.cpp b/STDLogSink.cpp
@@ -45,9 +45,16 @@ STDLogSink::STDLogSink(Severity min_severity)
 {
 	//Get the current display terminal width
 #ifndef _WIN32
-	struct winsize w;
-	ioctl(0, TIOCGWINSZ, &w);
-	m_termWidth = w.ws_col;
+	if(isatty(stdout))
+	{
+		struct winsize w;
+		ioctl(0, TIOCGWINSZ, &w);
+		m_termWidth = w.ws_col;
+	}
+	else
+	{
+		m_termWidth = 120;	//reasonable default width for file logging
+	}
 #else
 	CONSOLE_SCREEN_BUFFER_INFO csbi;
 	GetConsoleScreenBufferInfo(GetStdHandle(STD_OUTPUT_HANDLE), &csbi);
