Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Stop indenting stdout/stderr when running tests #19299

Merged
merged 1 commit into from
Nov 27, 2014

Conversation

nikomatsakis
Copy link
Contributor

This fixes a long-time irritant of mine. Inserting tabs causes M-x next-error to not work in emacs and seems to serve relatively little purpose in improving overall readability.

r? @brson

@pnkfelix
Copy link
Member

(i've considering hacking on the emacs regexp to allowing leading whitespace when seeking errors, but this PR is probably the better choice unless someone can argue for why indenting is necessary.)

alexcrichton added a commit to alexcrichton/rust that referenced this pull request Nov 27, 2014
This fixes a long-time irritant of mine. Inserting tabs causes M-x next-error to not work in emacs and seems to serve relatively little purpose in improving overall readability.

r? @brson
@bors bors merged commit 4671f7f into rust-lang:master Nov 27, 2014
@nikomatsakis nikomatsakis deleted the stop-indenting-test-output branch March 30, 2016 16:17
kennytm added a commit to kennytm/rust that referenced this pull request May 14, 2018
…crichton

Remove leftover tab in libtest outputs

This removes some tabs that were present in the output of libtest.

Related rust-lang#19299
Closes rust-lang#50362
Mark-Simulacrum added a commit to Mark-Simulacrum/rust that referenced this pull request May 17, 2018
…crichton

Remove leftover tab in libtest outputs

This removes some tabs that were present in the output of libtest.

Related rust-lang#19299
Closes rust-lang#50362
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants