encounter
/
dawn-cmake
mirror of https://github.com/encounter/dawn-cmake.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

formatter: Remove colon line prefix with no source 

If a diagnostic has no Source information, don't start the diagnostic line with a colon.

Bug: tint:282
Change-Id: I80c4103e31556b2769d4b4c2a98dce21a2e1c233
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/31662
Reviewed-by: dan sinclair <dsinclair@chromium.org>
Commit-Queue: Ben Clayton <bclayton@google.com>
This commit is contained in:
Ben Clayton 2020-11-03 18:27:29 +00:00 committed by Commit Bot service account
parent 06c892a6ec
commit 1a61e8613e
2 changed files with 20 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
src/diagnostic/formatter.cc
View File

@ -137,13 +137,17 @@ void Formatter::format(const Diagnostic& diag, State& state) const {
state.set_style({Color::kDefault, true});
bool emit_colon = true;
if (style_.print_file && src.file != nullptr && !src.file->path.empty()) {
state << src.file->path;
if (rng.begin.line > 0) {
state << ":" << rng.begin;
}
} else {
} else if (rng.begin.line > 0) {
state << rng.begin;
} else {
// No position infomation was printed, so don't start the line with a colon.
emit_colon = false;
}
if (style_.print_severity) {
switch (diag.severity) {
@ -157,11 +161,16 @@ void Formatter::format(const Diagnostic& diag, State& state) const {
default:
break;
}
state << " " << diag.severity;
state << " " << diag.severity << ": ";
// A colon was just printed, don't repeat it.
emit_colon = false;
}
state.set_style({Color::kDefault, true});
state << ": " << diag.message;
if (emit_colon) {
state << ": ";
}
state << diag.message;
if (style_.print_line && src.file != nullptr && rng.begin.line > 0) {
state.newline();

8
src/diagnostic/formatter_test.cc
View File

@ -53,6 +53,14 @@ TEST_F(DiagFormatterTest, Simple) {
ASSERT_EQ(expect, got);
}
TEST_F(DiagFormatterTest, SimpleNoSource) {
Formatter fmt{{false, false, false}};
Diagnostic diag{Severity::Info, Source{}, "no source!"};
auto got = fmt.format(List{diag});
auto* expect = "no source!";
ASSERT_EQ(expect, got);
}
TEST_F(DiagFormatterTest, WithFile) {
Formatter fmt{{true, false, false}};
auto got = fmt.format(List{diag_info, diag_warn, diag_err, diag_fatal});