{"action":"created","issue":{"url":"https://api.github.com/repos/dotnet/corefx/issues/1183","labels_url":"https://api.github.com/repos/dotnet/corefx/issues/1183/labels{/name}","comments_url":"https://api.github.com/repos/dotnet/corefx/issues/1183/comments","events_url":"https://api.github.com/repos/dotnet/corefx/issues/1183/events","html_url":"https://github.com/dotnet/corefx/pull/1183","id":63112015,"number":1183,"title":"TraceSource avoid allocating unused TraceEventCache","user":{"login":"MarkPflug","id":7776174,"avatar_url":"https://avatars.githubusercontent.com/u/7776174?v=3","gravatar_id":"","url":"https://api.github.com/users/MarkPflug","html_url":"https://github.com/MarkPflug","followers_url":"https://api.github.com/users/MarkPflug/followers","following_url":"https://api.github.com/users/MarkPflug/following{/other_user}","gists_url":"https://api.github.com/users/MarkPflug/gists{/gist_id}","starred_url":"https://api.github.com/users/MarkPflug/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MarkPflug/subscriptions","organizations_url":"https://api.github.com/users/MarkPflug/orgs","repos_url":"https://api.github.com/users/MarkPflug/repos","events_url":"https://api.github.com/users/MarkPflug/events{/privacy}","received_events_url":"https://api.github.com/users/MarkPflug/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/dotnet/corefx/labels/cla-signed","name":"cla-signed","color":"fef2c0"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":8,"created_at":"2015-03-19T23:08:16Z","updated_at":"2015-03-20T12:22:06Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/dotnet/corefx/pulls/1183","html_url":"https://github.com/dotnet/corefx/pull/1183","diff_url":"https://github.com/dotnet/corefx/pull/1183.diff","patch_url":"https://github.com/dotnet/corefx/pull/1183.patch"},"body":"Fix: The System.Diagnostics.TraceSource, TraceEvent and TraceData methods allocate a TraceEventCache object that might not be used if the switch is disabled or there are no listeners attached."},"comment":{"url":"https://api.github.com/repos/dotnet/corefx/issues/comments/84002422","html_url":"https://github.com/dotnet/corefx/pull/1183#issuecomment-84002422","issue_url":"https://api.github.com/repos/dotnet/corefx/issues/1183","id":84002422,"user":{"login":"stephentoub","id":2642209,"avatar_url":"https://avatars.githubusercontent.com/u/2642209?v=3","gravatar_id":"","url":"https://api.github.com/users/stephentoub","html_url":"https://github.com/stephentoub","followers_url":"https://api.github.com/users/stephentoub/followers","following_url":"https://api.github.com/users/stephentoub/following{/other_user}","gists_url":"https://api.github.com/users/stephentoub/gists{/gist_id}","starred_url":"https://api.github.com/users/stephentoub/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stephentoub/subscriptions","organizations_url":"https://api.github.com/users/stephentoub/orgs","repos_url":"https://api.github.com/users/stephentoub/repos","events_url":"https://api.github.com/users/stephentoub/events{/privacy}","received_events_url":"https://api.github.com/users/stephentoub/received_events","type":"User","site_admin":false},"created_at":"2015-03-20T12:22:06Z","updated_at":"2015-03-20T12:22:06Z","body":"@MarkPflug, thanks for adding the tests!  Normally we'd prefer that such tests be done separately, and first, so that we could see them passing first.  But in this case I think it's ok.  And just to be sure, I checked out your branch and verified that the tests pass with and without the product changes.  Other than my minor comments, LGTM."}}