{"action":"created","issue":{"url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/12470","labels_url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/12470/labels{/name}","comments_url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/12470/comments","events_url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/12470/events","html_url":"https://github.com/CleverRaven/Cataclysm-DDA/issues/12470","id":81064511,"number":12470,"title":"Recent changes to the lua-interface cause warnings/errors during compilation","user":{"login":"namarrgon","id":3209906,"avatar_url":"https://avatars.githubusercontent.com/u/3209906?v=3","gravatar_id":"","url":"https://api.github.com/users/namarrgon","html_url":"https://github.com/namarrgon","followers_url":"https://api.github.com/users/namarrgon/followers","following_url":"https://api.github.com/users/namarrgon/following{/other_user}","gists_url":"https://api.github.com/users/namarrgon/gists{/gist_id}","starred_url":"https://api.github.com/users/namarrgon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/namarrgon/subscriptions","organizations_url":"https://api.github.com/users/namarrgon/orgs","repos_url":"https://api.github.com/users/namarrgon/repos","events_url":"https://api.github.com/users/namarrgon/events{/privacy}","received_events_url":"https://api.github.com/users/namarrgon/received_events","type":"User","site_admin":false},"labels":[],"state":"closed","locked":false,"assignee":null,"milestone":null,"comments":3,"created_at":"2015-05-26T17:41:40Z","updated_at":"2015-05-27T04:24:25Z","closed_at":"2015-05-27T03:58:06Z","body":"PR #12451 introduced warnings during the build with LUA=1 that turn into errors with RELEASE=1 and so cause the build-process to fail. Apart from that there doesn't seem to be anything obviously broken.\r\n```\r\ng++ -DLOCALIZE -Wall -Wextra -g -D_GLIBCXX_DEBUG  --std=c++11 -MMD    -DLUA  -c src/catalua.cpp -o obj/catalua.o\r\nsrc/catalua.cpp: In instantiation of ‘static T& LuaValue<T>::get(lua_State*, int) [with T = point; lua_State = lua_State]’:\r\nsrc/lua/catabindings.cpp:1429:46:   required from here\r\nsrc/catalua.cpp:272:85: warning: cast to pointer from integer of different size [-Wint-to-pointer-cast]\r\n             return *(T*) luaL_error( L, \"First argument to function is not a class\" );\r\n                                                                                     ^\r\nsrc/catalua.cpp: In instantiation of ‘static T& LuaValue<T>::get(lua_State*, int) [with T = tripoint; lua_State = lua_State]’:\r\nsrc/lua/catabindings.cpp:2434:52:   required from here\r\n```\r\ngcc version 5.1.0 (GCC):\r\nhttps://gist.github.com/namarrgon/a3b19389247076eec5a1\r\n\r\nclang version 3.6.1 (tags/RELEASE_361/final):\r\n https://gist.github.com/namarrgon/18faacd31875879516e5"},"comment":{"url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/comments/105751518","html_url":"https://github.com/CleverRaven/Cataclysm-DDA/issues/12470#issuecomment-105751518","issue_url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/12470","id":105751518,"user":{"login":"narc0tiq","id":591573,"avatar_url":"https://avatars.githubusercontent.com/u/591573?v=3","gravatar_id":"","url":"https://api.github.com/users/narc0tiq","html_url":"https://github.com/narc0tiq","followers_url":"https://api.github.com/users/narc0tiq/followers","following_url":"https://api.github.com/users/narc0tiq/following{/other_user}","gists_url":"https://api.github.com/users/narc0tiq/gists{/gist_id}","starred_url":"https://api.github.com/users/narc0tiq/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/narc0tiq/subscriptions","organizations_url":"https://api.github.com/users/narc0tiq/orgs","repos_url":"https://api.github.com/users/narc0tiq/repos","events_url":"https://api.github.com/users/narc0tiq/events{/privacy}","received_events_url":"https://api.github.com/users/narc0tiq/received_events","type":"User","site_admin":false},"created_at":"2015-05-27T04:24:25Z","updated_at":"2015-05-27T04:24:25Z","body":"> Maybe the Pull Request builder should build a 64-bit binary?\r\n\r\nMakes sense to me -- it wasn't an option when initially set up, but we can do that now and the 64-bit builds do seem to be the more fragile ones. Done."}}