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

Reduce CPU cost of suppressed trace events #656

Merged
merged 2 commits into from Aug 2, 2018

Conversation

Projects
None yet
1 participant
@etschannen
Copy link
Contributor

etschannen commented Aug 2, 2018

Call all methods which could disable a trace event before it is initialized. In practice this means calling .error first, then .suppressFor, then all your details.

Fixed a bug where the trace event suppress backstop was counting events which were later suppressed through other mechanisms.

call all methods which could disable a trace event before it is initi…
…alized. In practice this means calling .error first, then .suppressFor, then all your details.

@etschannen etschannen requested a review from ajbeamon Aug 2, 2018

@etschannen etschannen merged commit 88ab07a into apple:release-6.0 Aug 2, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment