Auto merge of #51485 - estebank:dehighlight-secondary-msgs, r=GuillaumeGomez

Remove highlighting from secondary messages

Deemphasize the secondary messages so that all other highlights stand
out more.

<img width="684" alt="" src="https://user-images.githubusercontent.com/1606434/41261199-7b4fe96e-6d8f-11e8-8619-04d170617df2.png">
This commit is contained in:
bors 2018-07-21 23:50:28 +00:00
commit 0ad6179d3b
2 changed files with 11 additions and 4 deletions

View File

@ -963,6 +963,11 @@ impl EmitterWriter {
is_secondary: bool) is_secondary: bool)
-> io::Result<()> { -> io::Result<()> {
let mut buffer = StyledBuffer::new(); let mut buffer = StyledBuffer::new();
let header_style = if is_secondary {
Style::HeaderMsg
} else {
Style::MainHeaderMsg
};
if msp.primary_spans().is_empty() && msp.span_labels().is_empty() && is_secondary if msp.primary_spans().is_empty() && msp.span_labels().is_empty() && is_secondary
&& !self.short_message { && !self.short_message {
@ -973,7 +978,7 @@ impl EmitterWriter {
draw_note_separator(&mut buffer, 0, max_line_num_len + 1); draw_note_separator(&mut buffer, 0, max_line_num_len + 1);
let level_str = level.to_string(); let level_str = level.to_string();
if !level_str.is_empty() { if !level_str.is_empty() {
buffer.append(0, &level_str, Style::HeaderMsg); buffer.append(0, &level_str, Style::MainHeaderMsg);
buffer.append(0, ": ", Style::NoStyle); buffer.append(0, ": ", Style::NoStyle);
} }
self.msg_to_buffer(&mut buffer, msg, max_line_num_len, "note", None); self.msg_to_buffer(&mut buffer, msg, max_line_num_len, "note", None);
@ -989,10 +994,10 @@ impl EmitterWriter {
buffer.append(0, "]", Style::Level(level.clone())); buffer.append(0, "]", Style::Level(level.clone()));
} }
if !level_str.is_empty() { if !level_str.is_empty() {
buffer.append(0, ": ", Style::HeaderMsg); buffer.append(0, ": ", header_style);
} }
for &(ref text, _) in msg.iter() { for &(ref text, _) in msg.iter() {
buffer.append(0, text, Style::HeaderMsg); buffer.append(0, text, header_style);
} }
} }
@ -1521,7 +1526,7 @@ impl<'a> WritableDst<'a> {
} }
} }
Style::Quotation => {} Style::Quotation => {}
Style::OldSchoolNoteText | Style::HeaderMsg => { Style::OldSchoolNoteText | Style::MainHeaderMsg => {
spec.set_bold(true); spec.set_bold(true);
if cfg!(windows) { if cfg!(windows) {
spec.set_intense(true) spec.set_intense(true)
@ -1542,6 +1547,7 @@ impl<'a> WritableDst<'a> {
spec.set_fg(Some(Color::Blue)); spec.set_fg(Some(Color::Blue));
} }
} }
Style::HeaderMsg |
Style::NoStyle => {} Style::NoStyle => {}
Style::Level(lvl) => { Style::Level(lvl) => {
spec = lvl.color(); spec = lvl.color();

View File

@ -182,6 +182,7 @@ pub struct StyledString {
#[derive(Copy, Clone, Debug, PartialEq, Hash, RustcEncodable, RustcDecodable)] #[derive(Copy, Clone, Debug, PartialEq, Hash, RustcEncodable, RustcDecodable)]
pub enum Style { pub enum Style {
MainHeaderMsg,
HeaderMsg, HeaderMsg,
LineAndColumn, LineAndColumn,
LineNumber, LineNumber,