{"action":"created","issue":{"url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/3542","labels_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/3542/labels{/name}","comments_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/3542/comments","events_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/3542/events","html_url":"https://github.com/DynamoDS/Dynamo/pull/3542","id":54301587,"number":3542,"title":"Remove reference counting code","user":{"login":"ke-yu","id":2600379,"avatar_url":"https://avatars.githubusercontent.com/u/2600379?v=3","gravatar_id":"","url":"https://api.github.com/users/ke-yu","html_url":"https://github.com/ke-yu","followers_url":"https://api.github.com/users/ke-yu/followers","following_url":"https://api.github.com/users/ke-yu/following{/other_user}","gists_url":"https://api.github.com/users/ke-yu/gists{/gist_id}","starred_url":"https://api.github.com/users/ke-yu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ke-yu/subscriptions","organizations_url":"https://api.github.com/users/ke-yu/orgs","repos_url":"https://api.github.com/users/ke-yu/repos","events_url":"https://api.github.com/users/ke-yu/events{/privacy}","received_events_url":"https://api.github.com/users/ke-yu/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-14T09:39:23Z","updated_at":"2015-01-14T16:01:01Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/DynamoDS/Dynamo/pulls/3542","html_url":"https://github.com/DynamoDS/Dynamo/pull/3542","diff_url":"https://github.com/DynamoDS/Dynamo/pull/3542.diff","patch_url":"https://github.com/DynamoDS/Dynamo/pull/3542.patch"},"body":"This pull request removes reference counting related dead code. \r\n\r\nI've run unit tests to ensure no regression.\r\n\r\n@lukechurch , @junmendoza , please review/suggest if I should remove them or put them in compilation flag."},"comment":{"url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/comments/69938752","html_url":"https://github.com/DynamoDS/Dynamo/pull/3542#issuecomment-69938752","issue_url":"https://api.github.com/repos/DynamoDS/Dynamo/issues/3542","id":69938752,"user":{"login":"lukechurch","id":2978182,"avatar_url":"https://avatars.githubusercontent.com/u/2978182?v=3","gravatar_id":"","url":"https://api.github.com/users/lukechurch","html_url":"https://github.com/lukechurch","followers_url":"https://api.github.com/users/lukechurch/followers","following_url":"https://api.github.com/users/lukechurch/following{/other_user}","gists_url":"https://api.github.com/users/lukechurch/gists{/gist_id}","starred_url":"https://api.github.com/users/lukechurch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lukechurch/subscriptions","organizations_url":"https://api.github.com/users/lukechurch/orgs","repos_url":"https://api.github.com/users/lukechurch/repos","events_url":"https://api.github.com/users/lukechurch/events{/privacy}","received_events_url":"https://api.github.com/users/lukechurch/received_events","type":"User","site_admin":false},"created_at":"2015-01-14T16:01:01Z","updated_at":"2015-01-14T16:01:01Z","body":":+1: :+1: :+1:\r\n\r\nAm I correct in thinking that this is only part of the work? There look to be some lingering references to GCDotMethods hanging around, and the now always true compile flag of '#if GC_MARK_AND_SWEEP'\r\n\r\nThis all looks fairly safe to remove and most of it can be proven to be dead with Resharper, so I'd suggest straight removal.\r\n\r\nPlease consider syncing with @sharadkjaiswal or @RodRecker about exactly when we choose to bring this in to ensure it doesn't interfere with us extracting ourselves from the bugs from the refactor.\r\n\r\nLGTM.\r\n\r\n"}}