{"action":"created","issue":{"url":"https://api.github.com/repos/Oberon00/luabind/issues/29","labels_url":"https://api.github.com/repos/Oberon00/luabind/issues/29/labels{/name}","comments_url":"https://api.github.com/repos/Oberon00/luabind/issues/29/comments","events_url":"https://api.github.com/repos/Oberon00/luabind/issues/29/events","html_url":"https://github.com/Oberon00/luabind/pull/29","id":102665314,"number":29,"title":"[For discussion purpose] - Used unique_ptr instead of auto_ptr when c++11 is used.","user":{"login":"Bertram25","id":765917,"avatar_url":"https://avatars.githubusercontent.com/u/765917?v=3","gravatar_id":"","url":"https://api.github.com/users/Bertram25","html_url":"https://github.com/Bertram25","followers_url":"https://api.github.com/users/Bertram25/followers","following_url":"https://api.github.com/users/Bertram25/following{/other_user}","gists_url":"https://api.github.com/users/Bertram25/gists{/gist_id}","starred_url":"https://api.github.com/users/Bertram25/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Bertram25/subscriptions","organizations_url":"https://api.github.com/users/Bertram25/orgs","repos_url":"https://api.github.com/users/Bertram25/repos","events_url":"https://api.github.com/users/Bertram25/events{/privacy}","received_events_url":"https://api.github.com/users/Bertram25/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2015-08-23T22:05:29Z","updated_at":"2015-08-25T08:55:30Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Oberon00/luabind/pulls/29","html_url":"https://github.com/Oberon00/luabind/pull/29","diff_url":"https://github.com/Oberon00/luabind/pull/29.diff","patch_url":"https://github.com/Oberon00/luabind/pull/29.patch"},"body":"Hey @Oberon00 :D \r\n\r\n(CC @endoalir for info.)\r\n\r\nAfter some fiddling and some research, I manage to obtain a luabind version capable to compile without any warnings when using c++11, and that seems to run Valyria Tear rather fine, according to my latest tests... You can see the changes I made in this PR.\r\n\r\nYET, (because nice things never comes free), it seems those changes aren't perfect according to the following issue: https://github.com/HoverRace/HoverRace/issues/260 \r\n\r\nIt seems the following adapted commit is also needed: https://github.com/halmd-org/luaponte/commit/18537877740c3ca540f11a69160706d6e5891e8d because otherwise the following issue may happen: https://github.com/HoverRace/HoverRace/issues/259\r\n\r\nHowever, it seems HoverRace is using the main luabind repository and that other bugs may come into game, though. So I made this PR mainly to know what to do next in order to get a c++11-warning-free-and-functional luabind PR. Do you think my PR is enough, or do you have hints anywhere at how I should handle the issue around boost::shared_ptr vs std::shared_ptr?\r\n\r\nThanks a lot for your repository in any case and for your help, and best regards,"},"comment":{"url":"https://api.github.com/repos/Oberon00/luabind/issues/comments/134530745","html_url":"https://github.com/Oberon00/luabind/pull/29#issuecomment-134530745","issue_url":"https://api.github.com/repos/Oberon00/luabind/issues/29","id":134530745,"user":{"login":"fhoefling","id":2470626,"avatar_url":"https://avatars.githubusercontent.com/u/2470626?v=3","gravatar_id":"","url":"https://api.github.com/users/fhoefling","html_url":"https://github.com/fhoefling","followers_url":"https://api.github.com/users/fhoefling/followers","following_url":"https://api.github.com/users/fhoefling/following{/other_user}","gists_url":"https://api.github.com/users/fhoefling/gists{/gist_id}","starred_url":"https://api.github.com/users/fhoefling/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fhoefling/subscriptions","organizations_url":"https://api.github.com/users/fhoefling/orgs","repos_url":"https://api.github.com/users/fhoefling/repos","events_url":"https://api.github.com/users/fhoefling/events{/privacy}","received_events_url":"https://api.github.com/users/fhoefling/received_events","type":"User","site_admin":false},"created_at":"2015-08-25T08:55:30Z","updated_at":"2015-08-25T08:55:30Z","body":"Am 24.08.2015, 00:05 Uhr, schrieb Yohann Ferreira  \n<notifications@github.com>:\n\n> Hey @Oberon00 :D\n>\n> (CC @endoalir for info.)\n>\n> After some fiddling and some research, I manage to obtain a luabind  \n> version capable to compile without any warnings when using c++11, and  \n> that seems to run Valyria Tear rather fine, according to my latest  \n> tests... You can see the changes I made in this PR.\n>\n> YET, (because nice things never comes free), it seems those changes  \n> aren't perfect according to the following issue:  \n> https://github.com/HoverRace/HoverRace/issues/260\n>\n> It seems the following adapted commit is also needed:  \n> https://github.com/halmd-org/luaponte/commit/18537877740c3ca540f11a69160706d6e5891e8d  \n> because otherwise the following issue may happen:  \n> https://github.com/HoverRace/HoverRace/issues/259\n>\nIn HALMD, we've been using luabind (=luaponte) with C++11 since long  \nwithout any issues. Our version is perhaps a bit out-dated since I didn't  \nhave time to follow the recent developments in luabind, but it brings all  \nthe functionality we need (as of luabind 0.9.1) and works with the most  \nrecent Boost releases.\n\nI'd love to revert from luaponte to luabind and to use a library which is  \nsupported by a (small) community. Still I'm not sure which luabind branch  \nto start with, build-improvements seemed to be the most useful one for me  \nbut it hasn't been continued. I think the most urgent need is a kind of  \nrelease schedule and finally a new release of luabind (following the old  \n0.9.1). Featurewise, the various master branches (from different people)  \nare in a good shape as far as I can tell. However, just adding commits to  \n\"master\" is not enough, it does not tell how stable the current tip of the  \nbranch is. And stability is one of the most important criteria to make a  \nlarge project depend on a certain library.\n\nBTW, there is at least one failing test concerning \"out_value\" vs.  \n\"pure_out_value\", see commit a1199b5.\n\n\nRegards,\n\nFelix\n"}}