{"action":"created","issue":{"url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/10825","labels_url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/10825/labels{/name}","comments_url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/10825/comments","events_url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/10825/events","html_url":"https://github.com/CleverRaven/Cataclysm-DDA/pull/10825","id":53982546,"number":10825,"title":"Move it! 1 (Start moving player.h stuff to Character.h)","user":{"login":"i2amroy","id":4275617,"avatar_url":"https://avatars.githubusercontent.com/u/4275617?v=3","gravatar_id":"","url":"https://api.github.com/users/i2amroy","html_url":"https://github.com/i2amroy","followers_url":"https://api.github.com/users/i2amroy/followers","following_url":"https://api.github.com/users/i2amroy/following{/other_user}","gists_url":"https://api.github.com/users/i2amroy/gists{/gist_id}","starred_url":"https://api.github.com/users/i2amroy/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/i2amroy/subscriptions","organizations_url":"https://api.github.com/users/i2amroy/orgs","repos_url":"https://api.github.com/users/i2amroy/repos","events_url":"https://api.github.com/users/i2amroy/events{/privacy}","received_events_url":"https://api.github.com/users/i2amroy/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/labels/validated","name":"validated","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-01-11T05:34:05Z","updated_at":"2015-01-11T16:04:15Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/pulls/10825","html_url":"https://github.com/CleverRaven/Cataclysm-DDA/pull/10825","diff_url":"https://github.com/CleverRaven/Cataclysm-DDA/pull/10825.diff","patch_url":"https://github.com/CleverRaven/Cataclysm-DDA/pull/10825.patch"},"body":"Since these are pretty high PR-use files I'm doing it in smaller chunks. This moves a chunk of player+npc attributes (mostly mutation, bionic, and inventory stuff) to the Character level instead of the Player one as a precursor to getting NPC's to not depend on the player files and functions."},"comment":{"url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/comments/69499391","html_url":"https://github.com/CleverRaven/Cataclysm-DDA/pull/10825#issuecomment-69499391","issue_url":"https://api.github.com/repos/CleverRaven/Cataclysm-DDA/issues/10825","id":69499391,"user":{"login":"BevapDin","id":5095435,"avatar_url":"https://avatars.githubusercontent.com/u/5095435?v=3","gravatar_id":"","url":"https://api.github.com/users/BevapDin","html_url":"https://github.com/BevapDin","followers_url":"https://api.github.com/users/BevapDin/followers","following_url":"https://api.github.com/users/BevapDin/following{/other_user}","gists_url":"https://api.github.com/users/BevapDin/gists{/gist_id}","starred_url":"https://api.github.com/users/BevapDin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/BevapDin/subscriptions","organizations_url":"https://api.github.com/users/BevapDin/orgs","repos_url":"https://api.github.com/users/BevapDin/repos","events_url":"https://api.github.com/users/BevapDin/events{/privacy}","received_events_url":"https://api.github.com/users/BevapDin/received_events","type":"User","site_admin":false},"created_at":"2015-01-11T16:04:15Z","updated_at":"2015-01-11T16:04:15Z","body":"It might be better to keep the functions in src/mutation.cpp and just make them proper members of Character. The same \"system\" is used for ranged and melee combat withing the ranged.cpp and melee.cpp files. It seems a good way to make the character.cpp / player.cpp files less cluttered."}}