Commit 64045042 by Jim Stichnoth

Subzero: Fix a warning message in the browser build.

Due to a mistake in the logic in https://codereview.chromium.org/1903553004 , a warning is printed even when LogFilename is properly set to "-". BUG= https://bugs.chromium.org/p/nativeclient/issues/detail?id=4370 R=jpp@chromium.org Review URL: https://codereview.chromium.org/1911713002 .
parent 2655d962
...@@ -277,7 +277,9 @@ void BrowserCompileServer::startCompileThread(int ObjFD) { ...@@ -277,7 +277,9 @@ void BrowserCompileServer::startCompileThread(int ObjFD) {
InputStream = new llvm::QueueStreamer(); InputStream = new llvm::QueueStreamer();
bool LogStreamFailure = false; bool LogStreamFailure = false;
int LogFD = STDOUT_FILENO; int LogFD = STDOUT_FILENO;
if (getFlags().getLogFilename() == "/dev/stderr") { if (getFlags().getLogFilename() == "-") {
// Common case, do nothing.
} else if (getFlags().getLogFilename() == "/dev/stderr") {
LogFD = STDERR_FILENO; LogFD = STDERR_FILENO;
} else { } else {
LogStreamFailure = true; LogStreamFailure = true;
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment