From 8187141b4afd0bdacae9bed3adae8426690d5853 Mon Sep 17 00:00:00 2001 From: Nikolai Kosjar Date: Tue, 7 Mar 2017 16:12:42 +0100 Subject: [PATCH] C++: Avoid printing to stderr and calling exit() We are not a compiler. Change-Id: I57d8283759d2587b890ff9bc1ca0c173cd910683 Reviewed-by: hjk Reviewed-by: David Schulz --- .../3rdparty/cplusplus/TranslationUnit.cpp | 38 +------------------ src/libs/3rdparty/cplusplus/TranslationUnit.h | 1 - 2 files changed, 1 insertion(+), 38 deletions(-) diff --git a/src/libs/3rdparty/cplusplus/TranslationUnit.cpp b/src/libs/3rdparty/cplusplus/TranslationUnit.cpp index ea180af5bfd..257b2b97c79 100644 --- a/src/libs/3rdparty/cplusplus/TranslationUnit.cpp +++ b/src/libs/3rdparty/cplusplus/TranslationUnit.cpp @@ -446,25 +446,8 @@ void TranslationUnit::message(DiagnosticClient::Level level, unsigned index, con const StringLiteral *fileName = 0; getTokenPosition(index, &line, &column, &fileName); - if (DiagnosticClient *client = control()->diagnosticClient()) { + if (DiagnosticClient *client = control()->diagnosticClient()) client->report(level, fileName, line, column, format, args); - } else { - fprintf(stderr, "%s:%u: ", fileName->chars(), line); - const char *l = "error"; - if (level == DiagnosticClient::Warning) - l = "warning"; - else if (level == DiagnosticClient::Fatal) - l = "fatal"; - fprintf(stderr, "%s: ", l); - - vfprintf(stderr, format, args); - fputc('\n', stderr); - - showErrorLine(index, column, stderr); - } - - if (level == DiagnosticClient::Fatal) - exit(EXIT_FAILURE); } void TranslationUnit::warning(unsigned index, const char *format, ...) @@ -553,25 +536,6 @@ void TranslationUnit::releaseTokensAndComments() _comments = 0; } -void TranslationUnit::showErrorLine(unsigned index, unsigned column, FILE *out) -{ - unsigned lineOffset = _lineOffsets[findLineNumber(tokenAt(index).utf16charsBegin())]; - for (const char *cp = _firstSourceChar + lineOffset + 1; *cp && *cp != '\n'; ++cp) { - fputc(*cp, out); - } - fputc('\n', out); - - const char *end = _firstSourceChar + lineOffset + 1 + column - 1; - for (const char *cp = _firstSourceChar + lineOffset + 1; cp != end; ++cp) { - if (*cp != '\t') - fputc(' ', out); - else - fputc('\t', out); - } - fputc('^', out); - fputc('\n', out); -} - void TranslationUnit::resetAST() { delete _pool; diff --git a/src/libs/3rdparty/cplusplus/TranslationUnit.h b/src/libs/3rdparty/cplusplus/TranslationUnit.h index 5b28cbd6e24..81605575a54 100644 --- a/src/libs/3rdparty/cplusplus/TranslationUnit.h +++ b/src/libs/3rdparty/cplusplus/TranslationUnit.h @@ -174,7 +174,6 @@ private: unsigned findLineNumber(unsigned utf16charOffset) const; unsigned findColumnNumber(unsigned utf16CharOffset, unsigned lineNumber) const; PPLine findPreprocessorLine(unsigned utf16charOffset) const; - void showErrorLine(unsigned index, unsigned column, FILE *out); static const Token nullToken;