Skip to content

Recommend debug-logging instead of debug - #917

Merged
mark-i-m merged 1 commit into
rust-lang:masterfrom
jyn514:debug=true
Oct 8, 2020
Merged

mark-i-m merged 1 commit into
rust-lang:masterfrom
jyn514:debug=true

Conversation

@jyn514

@jyn514 jyn514 commented Oct 8, 2020 •

Copy link
Copy Markdown
Member

This makes the compiler faster to run without sacrificing too much
functionality. It still shows logging so contributors aren't confused by
debug! doing nothing.

This includes the changes from rust-lang/rust#77701.

cc @Mark-Simulacrum

This makes the compiler faster to running without sacrificing too much
performance. It still shows logging so contributors aren't confused by
`debug!` doing nothing.
@jyn514 jyn514 added the S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content label Oct 8, 2020
@Mark-Simulacrum

Copy link
Copy Markdown
Member

Looks good to me.

@jyn514

jyn514 commented Oct 8, 2020

Copy link
Copy Markdown
Member Author

@mark-i-m while you're working on the dev-guide, could I snag a review? ;)

@mark-i-m
mark-i-m merged commit d1e314f into rust-lang:master Oct 8, 2020
@jyn514
jyn514 deleted the debug=true branch October 8, 2020 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-review Status: this PR is waiting for a reviewer to verify its content

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants