)]}'
{"swift/obj/reconstructor.py":[{"author":{"_account_id":10100,"name":"Ian Adams","email":"ian.adams@evault.com","username":"kelanis"},"change_message_id":"8bd7dfeb9bd626517f48cfa7e8fe2fc98b1b4ce1","unresolved":false,"context_lines":[{"line_number":340,"context_line":"        for node in part_nodes:"},{"line_number":341,"context_line":"            if node[\u0027id\u0027] \u003d\u003d local_id:"},{"line_number":342,"context_line":"                handoff_node \u003d False"},{"line_number":343,"context_line":"                index \u003d part_nodes.index(node)"},{"line_number":344,"context_line":"                left \u003d part_nodes[part_nodes.index(node) - 1:"},{"line_number":345,"context_line":"                                  part_nodes.index(node)]"},{"line_number":346,"context_line":"                right \u003d part_nodes[part_nodes.index(node) + 1:"}],"source_content_type":"text/x-python","patch_set":1,"id":"9aa7fdbe_b4dd38e2","line":343,"updated":"2014-10-28 17:09:01.000000000","message":"I could be totally missing something, but is all the slicing necessary? Maybe doing a little modulo arithmetic to get the left/right to wrapping around might be a bit cleaner to reason about? \n\nAlso, if we can avoid the slicing as those make copies of the lists. Not a huge thing, but a nice little bonus if we can trim it out.\n\nJust thinking out loud.","commit_id":"c936c6af105d48d80ce1b2c7d498135a7079ceba"},{"author":{"_account_id":7479,"name":"paul luse","email":"paul.e.luse@intel.com","username":"peluse"},"change_message_id":"33a00eee67b31cd43817813528d2e7880bd852be","unresolved":false,"context_lines":[{"line_number":340,"context_line":"        for node in part_nodes:"},{"line_number":341,"context_line":"            if node[\u0027id\u0027] \u003d\u003d local_id:"},{"line_number":342,"context_line":"                handoff_node \u003d False"},{"line_number":343,"context_line":"                index \u003d part_nodes.index(node)"},{"line_number":344,"context_line":"                left \u003d part_nodes[part_nodes.index(node) - 1:"},{"line_number":345,"context_line":"                                  part_nodes.index(node)]"},{"line_number":346,"context_line":"                right \u003d part_nodes[part_nodes.index(node) + 1:"}],"source_content_type":"text/x-python","patch_set":1,"id":"5a890539_13950233","line":343,"in_reply_to":"9aa7fdbe_b4dd38e2","updated":"2014-11-11 16:33:56.000000000","message":"open to an alternate suggested implement ion :)","commit_id":"c936c6af105d48d80ce1b2c7d498135a7079ceba"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"cedbeb036853c1c10d387d8af975d68f48335c7b","unresolved":false,"context_lines":[{"line_number":342,"context_line":"                handoff_node \u003d False"},{"line_number":343,"context_line":"                index \u003d part_nodes.index(node)"},{"line_number":344,"context_line":"                left \u003d part_nodes[part_nodes.index(node) - 1:"},{"line_number":345,"context_line":"                                  part_nodes.index(node)]"},{"line_number":346,"context_line":"                right \u003d part_nodes[part_nodes.index(node) + 1:"},{"line_number":347,"context_line":"                                   part_nodes.index(node) + 2]"},{"line_number":348,"context_line":"                if index \u003d\u003d 0:"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a961159_1736c803","line":345,"updated":"2014-12-17 02:16:52.000000000","message":"I think, we can use \"index\" variable 1 line above directly. (Able to avoid to call part_nodes.index())","commit_id":"60e2ea62a4e335688fbad0973f52a2a97a8ddf1c"},{"author":{"_account_id":4608,"name":"Kota Tsuyuzaki","email":"bloodeagle40234@gmail.com","username":"tsuyuzaki-kota"},"change_message_id":"cedbeb036853c1c10d387d8af975d68f48335c7b","unresolved":false,"context_lines":[{"line_number":346,"context_line":"                right \u003d part_nodes[part_nodes.index(node) + 1:"},{"line_number":347,"context_line":"                                   part_nodes.index(node) + 2]"},{"line_number":348,"context_line":"                if index \u003d\u003d 0:"},{"line_number":349,"context_line":"                    left \u003d part_nodes[len(part_nodes) - 1:]"},{"line_number":350,"context_line":"                elif index \u003d\u003d len(part_nodes) - 1:"},{"line_number":351,"context_line":"                    right \u003d part_nodes[:1]"},{"line_number":352,"context_line":"        if not handoff_node:"}],"source_content_type":"text/x-python","patch_set":4,"id":"3a961159_e981644e","line":349,"updated":"2014-12-17 02:16:52.000000000","message":"To simplify, we can code like as \"part_nodes[-1:]\" if you need last one item in the list.","commit_id":"60e2ea62a4e335688fbad0973f52a2a97a8ddf1c"}]}