GitHub PullRequestReviewCommentEvent at 2014-04-16 08:35:08
Repository: cmssw
Actor: arizzi
Payload: {"comment":{"url":"https://api.github.com/repos/cms-sw/cmssw/pulls/comments/11674340","id":11674340,"diff_hunk":"@@ -0,0 +1,140 @@\n+//\n+// $Id: PATElectronSlimmer.cc,v 1.1 2011/03/24 18:45:45 mwlebour Exp $\n+//\n+\n+/**\n+  \\class    pat::PATElectronSlimmer PATElectronSlimmer.h \"PhysicsTools/PatAlgos/interface/PATElectronSlimmer.h\"\n+  \\brief    Matcher of reconstructed objects to L1 Muons \n+            \n+  \\author   Giovanni Petrucciani\n+  \\version  $Id: PATElectronSlimmer.cc,v 1.1 2011/03/24 18:45:45 mwlebour Exp $\n+*/\n+#define private public\n+#define protected public","path":"PhysicsTools/PatAlgos/plugins/PATElectronSlimmer.cc","position":null,"original_position":13,"commit_id":"7671a083ce30613cd9c8490d4c6d26dab1fc8945","original_commit_id":"cd35ab44933217c8df512507ccd1e6def006fb1f","user":{"login":"arizzi","id":5022358,"avatar_url":"https://avatars.githubusercontent.com/u/5022358?","gravatar_id":"691e33f77942fb889afcb7c642a32e7b","url":"https://api.github.com/users/arizzi","html_url":"https://github.com/arizzi","followers_url":"https://api.github.com/users/arizzi/followers","following_url":"https://api.github.com/users/arizzi/following{/other_user}","gists_url":"https://api.github.com/users/arizzi/gists{/gist_id}","starred_url":"https://api.github.com/users/arizzi/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/arizzi/subscriptions","organizations_url":"https://api.github.com/users/arizzi/orgs","repos_url":"https://api.github.com/users/arizzi/repos","events_url":"https://api.github.com/users/arizzi/events{/privacy}","received_events_url":"https://api.github.com/users/arizzi/received_events","type":"User","site_admin":false},"body":"yes, I think this was in our slides from last week. We need the objects to have setters for data members. What are the policies on friend classes in CMSSW?","created_at":"2014-04-16T08:35:08Z","updated_at":"2014-04-16T08:35:08Z","html_url":"https://github.com/cms-sw/cmssw/pull/3253#discussion_r11674340","pull_request_url":"https://api.github.com/repos/cms-sw/cmssw/pulls/3253","_links":{"self":{"href":"https://api.github.com/repos/cms-sw/cmssw/pulls/comments/11674340"},"html":{"href":"https://github.com/cms-sw/cmssw/pull/3253#discussion_r11674340"},"pull_request":{"href":"https://api.github.com/repos/cms-sw/cmssw/pulls/3253"}}}}