{"action":"created","issue":{"url":"https://api.github.com/repos/ManyUniverseAna/RooMUHistos/issues/13","labels_url":"https://api.github.com/repos/ManyUniverseAna/RooMUHistos/issues/13/labels{/name}","comments_url":"https://api.github.com/repos/ManyUniverseAna/RooMUHistos/issues/13/comments","events_url":"https://api.github.com/repos/ManyUniverseAna/RooMUHistos/issues/13/events","html_url":"https://github.com/ManyUniverseAna/RooMUHistos/issues/13","id":64365997,"number":13,"title":"Very strongly consider removing Boost","user":{"login":"gnperdue","id":1294053,"avatar_url":"https://avatars.githubusercontent.com/u/1294053?v=3","gravatar_id":"","url":"https://api.github.com/users/gnperdue","html_url":"https://github.com/gnperdue","followers_url":"https://api.github.com/users/gnperdue/followers","following_url":"https://api.github.com/users/gnperdue/following{/other_user}","gists_url":"https://api.github.com/users/gnperdue/gists{/gist_id}","starred_url":"https://api.github.com/users/gnperdue/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gnperdue/subscriptions","organizations_url":"https://api.github.com/users/gnperdue/orgs","repos_url":"https://api.github.com/users/gnperdue/repos","events_url":"https://api.github.com/users/gnperdue/events{/privacy}","received_events_url":"https://api.github.com/users/gnperdue/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-03-25T20:32:47Z","updated_at":"2015-03-25T21:21:31Z","closed_at":null,"body":"Do we need to include Boost? Extra dependencies need to be carefully justified."},"comment":{"url":"https://api.github.com/repos/ManyUniverseAna/RooMUHistos/issues/comments/86221820","html_url":"https://github.com/ManyUniverseAna/RooMUHistos/issues/13#issuecomment-86221820","issue_url":"https://api.github.com/repos/ManyUniverseAna/RooMUHistos/issues/13","id":86221820,"user":{"login":"gnperdue","id":1294053,"avatar_url":"https://avatars.githubusercontent.com/u/1294053?v=3","gravatar_id":"","url":"https://api.github.com/users/gnperdue","html_url":"https://github.com/gnperdue","followers_url":"https://api.github.com/users/gnperdue/followers","following_url":"https://api.github.com/users/gnperdue/following{/other_user}","gists_url":"https://api.github.com/users/gnperdue/gists{/gist_id}","starred_url":"https://api.github.com/users/gnperdue/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/gnperdue/subscriptions","organizations_url":"https://api.github.com/users/gnperdue/orgs","repos_url":"https://api.github.com/users/gnperdue/repos","events_url":"https://api.github.com/users/gnperdue/events{/privacy}","received_events_url":"https://api.github.com/users/gnperdue/received_events","type":"User","site_admin":false},"created_at":"2015-03-25T21:21:31Z","updated_at":"2015-03-25T21:21:31Z","body":"There is `tr1::shared_ptr` (widely used in the Minerva DAQ) while we wait and dream for C++11 ...\n\npax\nGabe\n\n\n> On Mar 25, 2015, at 4:20 PM, iambriantice <notifications@github.com> wrote:\n> \n> The justification is just to protect ourselves from memory leaks. But this isn’t particularly difficult now.\n> It exists only in MUPlotter since the persisted classes have explicit ownership and ROOT’s streamer struggled with shared_ptr the last time I tried.\n> \n> You two have more experience in how experiments use external packages. If you think removing dependence on boost would help make RooMUHistos easier to adopt, then they can be replaced with carefully deleted bare pointers.\n> \n> If c++11 were widely used then the definition of TH1Ptr could use std::shared_ptr instead of boost::shared_ptr, which would nicely solve both issues.\n> \n> —————————————\n> Brian G. Tice\n> Argonne National Lab\n> 630-252-3057 - ANL\n> 630-840-5708 - FNAL\n> —————————————\n> \n> \n> \n> \n> \n> \n> \n> \n> > On Mar 25, 2015, at 3:51 PM, Julia Yarba <notifications@github.com> wrote:\n> > \n> > We're using boost::shared_ptr in the MUPlotter. Not in other classes.\n> > We can probably go around with it.\n> > But I'd leave it up to Brian to comment how much it's really needed.\n> > \n> > —\n> > Reply to this email directly or view it on GitHub <https://github.com/ManyUniverseAna/RooMUHistos/issues/13#issuecomment-86211757>.\n> > \n> \n> —\n> Reply to this email directly or view it on GitHub <https://github.com/ManyUniverseAna/RooMUHistos/issues/13#issuecomment-86221553>.\n> \n\n"}}