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

Disable debug logging in release builds #8229

Merged
merged 1 commit into from Oct 28, 2015
Merged

Conversation

@mbrubeck
Copy link
Contributor

mbrubeck commented Oct 27, 2015

This uses the new release_max_level feature added in rust-lang/log#58 to turn the debug! and trace! macros into no-ops in optimized builds.

r? @pcwalton (or anyone else who wants it)

Review on Reviewable

@bors-servo
Copy link
Contributor

bors-servo commented Oct 28, 2015

The latest upstream changes (presumably #8241) made this pull request unmergeable. Please resolve the merge conflicts.

This uses the new `release_max_level` feature added in
rust-lang/log#58 to turn the `debug!` and `trace!` macros into no-ops
in optimized builds.
@mbrubeck mbrubeck force-pushed the mbrubeck:debug-log branch from 6b9908c to ccecf3f Oct 28, 2015
@larsbergstrom
Copy link
Contributor

larsbergstrom commented Oct 28, 2015

@bors-servo
Copy link
Contributor

bors-servo commented Oct 28, 2015

📌 Commit ccecf3f has been approved by larsbergstrom

@bors-servo
Copy link
Contributor

bors-servo commented Oct 28, 2015

Testing commit ccecf3f with merge 9501564...

bors-servo added a commit that referenced this pull request Oct 28, 2015
Disable debug logging in release builds

This uses the new `release_max_level` feature added in rust-lang/log#58 to turn the `debug!` and `trace!` macros into no-ops in optimized builds.

r? @pcwalton (or anyone else who wants it)

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8229)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Oct 28, 2015

@bors-servo bors-servo merged commit ccecf3f into servo:master Oct 28, 2015
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
homu Test successful
Details
@mbrubeck mbrubeck deleted the mbrubeck:debug-log branch May 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.