{"push_id":1198745750,"size":19,"distinct_size":0,"ref":"refs/heads/master","head":"4654cd06ed73a01ea6d307703a65ffa76b7bd260","before":"15beb2ffa6fb1dfe4ec90cbf82a6315d4011efcb","commits":[{"sha":"131258da93d6aab5db5683d0cacd1f637dea6b73","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Add some whitespace/comments through the RandomPlacement config\nparsing, and take out an unnecessary jumping out/in of block.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/131258da93d6aab5db5683d0cacd1f637dea6b73"},{"sha":"b3ea454be8b4001220ec69b318e13d3dcb95440f","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Switch to using local vars rather than the global Junk's in this\nparsing, especially since 3/5 of them weren't even Junk.  Tweak debug\nmessages, and use an intermediate var to avoid rerepeating ourselves.\nNo functional change.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/b3ea454be8b4001220ec69b318e13d3dcb95440f"},{"sha":"f8f0d30905622c6a710ccaa94a634f7ce6b15169","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Switch to local geometry vars and an intermediate for the\nMaxWindowSize parsing.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/f8f0d30905622c6a710ccaa94a634f7ce6b15169"},{"sha":"879aafd17033949cdac1946ccddcbf0a1e6dd19d","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Don't need this intermediate var for one use.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/879aafd17033949cdac1946ccddcbf0a1e6dd19d"},{"sha":"5bb070a71a69c19f74dfe46625601a043942a260","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Use local vars for these X/Y coords, instead of abusing global Junk's\nfor values that aren't Junk.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/5bb070a71a69c19f74dfe46625601a043942a260"},{"sha":"9c50f3279a4168ed757c5aaa5aed840c48e27191","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Don't initialize Junk vars, they're junk.  Also on BSS, so they're\nzero anyway.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/9c50f3279a4168ed757c5aaa5aed840c48e27191"},{"sha":"030f303611c20c1e2b161ad06bd9422389d4305e","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Use a real var instead of the Junk here for a value we're actually\ndoing stuff with.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/030f303611c20c1e2b161ad06bd9422389d4305e"},{"sha":"7f5cf21fc06df0be98682bf578a2a97bd933b48c","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Rewrap line and make indent","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/7f5cf21fc06df0be98682bf578a2a97bd933b48c"},{"sha":"2ae8caa872cb717002faaf7cb9380e854ea27d59","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Use local var for this non-junk Root too.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/2ae8caa872cb717002faaf7cb9380e854ea27d59"},{"sha":"44800e4ba4f15948f1590bbf0b810de04f01e5d7","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Use the global junk vars here instead of shadowing them with\nalmost-but-not-quite same named locals.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/44800e4ba4f15948f1590bbf0b810de04f01e5d7"},{"sha":"c201ddc3eef2c2d66155afa1b5019ad7b0cc1708","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Replace places where we're actually using a Junk var with real local\nvars in events.c.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/c201ddc3eef2c2d66155afa1b5019ad7b0cc1708"},{"sha":"b2a7709368afd98c16f059c3c50e44f55b316b20","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Note on a huge block that it should be moved to an external func.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/b2a7709368afd98c16f059c3c50e44f55b316b20"},{"sha":"512fc4314e7922118c301e12f3dda17897bfe799","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Instead of storing a value into DragBW, just to assign it to JunkBW,\nand then using JunkBW in tons of real calculations, just use a local\nvar for the whole stupid thing.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/512fc4314e7922118c301e12f3dda17897bfe799"},{"sha":"dee5986511163eb3b2cc5e635094ddce2c692835","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Local vars instead of junk.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/dee5986511163eb3b2cc5e635094ddce2c692835"},{"sha":"8389bcf51b74dbd25286f693a1a4ff556546f401","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Pull the body of f.move and friends into a separate function, as it's\nso long to have inline.  A few of the vars in ExecuteFunction were\nonly used here, so move them; some others were otherwise only used in\nlimited places, so localize them too.  Switch back to setting DrawBW\nas an intermediate step in the code while I'm at it; it may be needed\nin the event stuff anyway.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/8389bcf51b74dbd25286f693a1a4ff556546f401"},{"sha":"942aeac3a6221915aa59a47ae4d02f9255413097","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Tpyo.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/942aeac3a6221915aa59a47ae4d02f9255413097"},{"sha":"d543dba709fc1317d371dde771e0647bdb189349","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Rewrap.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/d543dba709fc1317d371dde771e0647bdb189349"},{"sha":"70df8f36d5e6c09704f60eb15613c0fb37801c56","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"Add comments about the usage of Junk vars, and collapse their\ndefinitions a little.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/70df8f36d5e6c09704f60eb15613c0fb37801c56"},{"sha":"4654cd06ed73a01ea6d307703a65ffa76b7bd260","author":{"name":"Matthew Fuller","email":"bb582feba4eeda24b1acbc7475138cee2c3f0636@over-yonder.net"},"message":"More misc cleanups.\n\n- Replace Junk vars with real vars in places the values are actually\n  used.  This should be complete; remaining uses should all actually be\n  junk.  Consideration was given to moving them all to local vars and\n  eliminating the globals, but that probably makes things worse overall.\n\n- Move a large block of inline code in f. handling to a separate\n  function.  There are probably more of the same that are worth doing.\n\n- A few other small adjustments nearby.","distinct":false,"url":"https://api.github.com/repos/fullermd/ctwm-mirror/commits/4654cd06ed73a01ea6d307703a65ffa76b7bd260"}]}
{"actor":{"display_login":"fullermd"}}