{"action":"created","issue":{"url":"https://api.github.com/repos/d3athrow/vgstation13/issues/3551","labels_url":"https://api.github.com/repos/d3athrow/vgstation13/issues/3551/labels{/name}","comments_url":"https://api.github.com/repos/d3athrow/vgstation13/issues/3551/comments","events_url":"https://api.github.com/repos/d3athrow/vgstation13/issues/3551/events","html_url":"https://github.com/d3athrow/vgstation13/pull/3551","id":63714507,"number":3551,"title":"Update window code","user":{"login":"dylanstrategie","id":6137403,"avatar_url":"https://avatars.githubusercontent.com/u/6137403?v=3","gravatar_id":"","url":"https://api.github.com/users/dylanstrategie","html_url":"https://github.com/dylanstrategie","followers_url":"https://api.github.com/users/dylanstrategie/followers","following_url":"https://api.github.com/users/dylanstrategie/following{/other_user}","gists_url":"https://api.github.com/users/dylanstrategie/gists{/gist_id}","starred_url":"https://api.github.com/users/dylanstrategie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/dylanstrategie/subscriptions","organizations_url":"https://api.github.com/users/dylanstrategie/orgs","repos_url":"https://api.github.com/users/dylanstrategie/repos","events_url":"https://api.github.com/users/dylanstrategie/events{/privacy}","received_events_url":"https://api.github.com/users/dylanstrategie/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/d3athrow/vgstation13/labels/Conflicts","name":"Conflicts","color":"d4c5f9"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":4,"created_at":"2015-03-23T12:49:08Z","updated_at":"2015-03-25T11:36:09Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/d3athrow/vgstation13/pulls/3551","html_url":"https://github.com/d3athrow/vgstation13/pull/3551","diff_url":"https://github.com/d3athrow/vgstation13/pull/3551.diff","patch_url":"https://github.com/d3athrow/vgstation13/pull/3551.patch"},"body":"That code was old and fucky, goddamn\r\n\r\n- Span, formatting, newline, visible_message, yeah\r\n- Separate sprites for window panes and full windows. Might need a bit more refining still, but that's that\r\n- Remove deprecated commented silicate code. If anyone ever wants to include something akin to silicate, they would benefit from just recoding it from scratch. An item that adds health and potentially changes sprite color can't be that hard to make\r\n- Reduce normal window health by 4 to 10. Why would you use plain windows\r\n- Remove stupid paths that caused shitty inheritance. window/basic and window/full/basic were dumb as fucking shit and no longer exist (both transferred to their respective tree parent, for whatever variables needed transplanting, so 2-3). All full window types under full/ all reinforced window types under reinforced/, all in that order\r\n- Add in more examine() information (construction state and health)\r\n- Add in healthcheck() to replace destroy(), hit() and all that logging and variable juggling bullshit in about every proc where the window can be damaged\r\n- ex_act(), blob_act(), meteorhit() now deplete health only and no longer magically delete or Destroy() windows. Looks a bit ugly on ex_act 3 since reinforced windows can't be destroyed but grilles always are, and plasma windows still don't block explosions even if they're now almost bomb-proof, but that's progress\r\n- Remove window damage on passive grab window \"slam\", massively increase mob damage on neck/kill window slam (if you let yourself get grabbed that far, you deserve to die)\r\n- Change construction from that ugly variable juggling to a fancy and functional switch() construction code. Include bitflags and deconstruction safety\r\n- All files that were modified outside of window.dm and fullwindow.dm are compatibility fixes\r\n\r\nTry as I might, I couldn't address #3515. It appears to be fundamentally fucked due to the way pushing objects (you enter the window's tile) and pulling objects (you cannot reach the other side of the window, ever) works. Maybe someone else will manage it"},"comment":{"url":"https://api.github.com/repos/d3athrow/vgstation13/issues/comments/85992255","html_url":"https://github.com/d3athrow/vgstation13/pull/3551#issuecomment-85992255","issue_url":"https://api.github.com/repos/d3athrow/vgstation13/issues/3551","id":85992255,"user":{"login":"clusterfack","id":8516830,"avatar_url":"https://avatars.githubusercontent.com/u/8516830?v=3","gravatar_id":"","url":"https://api.github.com/users/clusterfack","html_url":"https://github.com/clusterfack","followers_url":"https://api.github.com/users/clusterfack/followers","following_url":"https://api.github.com/users/clusterfack/following{/other_user}","gists_url":"https://api.github.com/users/clusterfack/gists{/gist_id}","starred_url":"https://api.github.com/users/clusterfack/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clusterfack/subscriptions","organizations_url":"https://api.github.com/users/clusterfack/orgs","repos_url":"https://api.github.com/users/clusterfack/repos","events_url":"https://api.github.com/users/clusterfack/events{/privacy}","received_events_url":"https://api.github.com/users/clusterfack/received_events","type":"User","site_admin":false},"created_at":"2015-03-25T11:36:09Z","updated_at":"2015-03-25T11:36:09Z","body":"All them merges"}}