)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":20354,"name":"Shoham Peller","email":"shohamp@gmail.com","username":"shoham.peller"},"change_message_id":"2b003ef4b00cf82816f9bd3e246e83022c675fef","unresolved":false,"context_lines":[{"line_number":12,"context_line":"will insert it first to the cmd parsing list."},{"line_number":13,"context_line":"When this happends, trying to enter the command \u0027foo bar\u0027 will"},{"line_number":14,"context_line":"envoke the command \u0027foo\u0027 with position argument \u0027bar\u0027, and not"},{"line_number":15,"context_line":"the scond command \u0027foo bar\u0027."},{"line_number":16,"context_line":"By iterating over the dict in a sorted fashion, \u0027foo bar\u0027 will"},{"line_number":17,"context_line":"be parsed before the \u0027foo\u0027 command and the right command will be"},{"line_number":18,"context_line":"envoked."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fa7ab95a_fc75e6f9","line":15,"updated":"2016-08-29 12:41:03.000000000","message":"*second","commit_id":"5d5289f857a1ad151eff69731ad6b9f9181758db"}],"cliff/commandmanager.py":[{"author":{"_account_id":20354,"name":"Shoham Peller","email":"shohamp@gmail.com","username":"shoham.peller"},"change_message_id":"d915c5081fa0563385d99150ca83fb6b9e7ed8b9","unresolved":false,"context_lines":[{"line_number":69,"context_line":"                raise ValueError(\u0027Invalid command %r\u0027 % name)"},{"line_number":70,"context_line":"            next_val \u003d search_args.pop(0)"},{"line_number":71,"context_line":"            name \u003d \u0027%s %s\u0027 % (name, next_val) if name else next_val"},{"line_number":72,"context_line":"            if name in sorted(self.commands, key\u003dlen, reverse\u003dTrue):"},{"line_number":73,"context_line":"                cmd_ep \u003d self.commands[name]"},{"line_number":74,"context_line":"                if hasattr(cmd_ep, \u0027resolve\u0027):"},{"line_number":75,"context_line":"                    cmd_factory \u003d cmd_ep.resolve()"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_fc9166f2","line":72,"updated":"2016-08-29 12:45:30.000000000","message":"isn\u0027t key\u003dlen is problematic? doesn\u0027t it mean that the sorted function only compares len and not alphabetically?","commit_id":"5d5289f857a1ad151eff69731ad6b9f9181758db"},{"author":{"_account_id":20424,"name":"Eyal Posener","email":"eyal@stratoscale.com","username":"eyal"},"change_message_id":"09183e2e70676539593910e3cd3782e1cb7c1e56","unresolved":false,"context_lines":[{"line_number":69,"context_line":"                raise ValueError(\u0027Invalid command %r\u0027 % name)"},{"line_number":70,"context_line":"            next_val \u003d search_args.pop(0)"},{"line_number":71,"context_line":"            name \u003d \u0027%s %s\u0027 % (name, next_val) if name else next_val"},{"line_number":72,"context_line":"            if name in sorted(self.commands, key\u003dlen, reverse\u003dTrue):"},{"line_number":73,"context_line":"                cmd_ep \u003d self.commands[name]"},{"line_number":74,"context_line":"                if hasattr(cmd_ep, \u0027resolve\u0027):"},{"line_number":75,"context_line":"                    cmd_factory \u003d cmd_ep.resolve()"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa7ab95a_0f688a86","line":72,"in_reply_to":"fa7ab95a_fc9166f2","updated":"2016-08-29 12:48:16.000000000","message":"len is what i care about.\nso \u0027foo bar\u0027 will be before \u0027foo\u0027","commit_id":"5d5289f857a1ad151eff69731ad6b9f9181758db"},{"author":{"_account_id":20354,"name":"Shoham Peller","email":"shohamp@gmail.com","username":"shoham.peller"},"change_message_id":"3a9d7e133692f024018457764443144e78ce15d6","unresolved":false,"context_lines":[{"line_number":69,"context_line":"                raise ValueError(\u0027Invalid command %r\u0027 % name)"},{"line_number":70,"context_line":"            next_val \u003d search_args.pop(0)"},{"line_number":71,"context_line":"            name \u003d \u0027%s %s\u0027 % (name, next_val) if name else next_val"},{"line_number":72,"context_line":"            if name in sorted(self.commands, key\u003dlen, reverse\u003dTrue):"},{"line_number":73,"context_line":"                cmd_ep \u003d self.commands[name]"},{"line_number":74,"context_line":"                if hasattr(cmd_ep, \u0027resolve\u0027):"},{"line_number":75,"context_line":"                    cmd_factory \u003d cmd_ep.resolve()"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_4f3372ab","line":72,"updated":"2016-08-29 12:47:31.000000000","message":"isn\u0027t key\u003dlen is problematic? doesn\u0027t it mean that the sorted function only compares len and not alphabetically?","commit_id":"6fa9ea8d44f2ea179fd951dc69dc36dbb87ad9b0"},{"author":{"_account_id":20354,"name":"Shoham Peller","email":"shohamp@gmail.com","username":"shoham.peller"},"change_message_id":"bf02da9d085c6b277e2b8e4b33f0fbe0acbba835","unresolved":false,"context_lines":[{"line_number":69,"context_line":"                raise ValueError(\u0027Invalid command %r\u0027 % name)"},{"line_number":70,"context_line":"            next_val \u003d search_args.pop(0)"},{"line_number":71,"context_line":"            name \u003d \u0027%s %s\u0027 % (name, next_val) if name else next_val"},{"line_number":72,"context_line":"            if name in sorted(self.commands, key\u003dlen, reverse\u003dTrue):"},{"line_number":73,"context_line":"                cmd_ep \u003d self.commands[name]"},{"line_number":74,"context_line":"                if hasattr(cmd_ep, \u0027resolve\u0027):"},{"line_number":75,"context_line":"                    cmd_factory \u003d cmd_ep.resolve()"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_cfda22e8","line":72,"in_reply_to":"fa7ab95a_4f329260","updated":"2016-08-29 12:53:16.000000000","message":"in the next line for example","commit_id":"6fa9ea8d44f2ea179fd951dc69dc36dbb87ad9b0"},{"author":{"_account_id":20354,"name":"Shoham Peller","email":"shohamp@gmail.com","username":"shoham.peller"},"change_message_id":"ff249f4e3a426920e1c754ee0feb619f37199ad7","unresolved":false,"context_lines":[{"line_number":69,"context_line":"                raise ValueError(\u0027Invalid command %r\u0027 % name)"},{"line_number":70,"context_line":"            next_val \u003d search_args.pop(0)"},{"line_number":71,"context_line":"            name \u003d \u0027%s %s\u0027 % (name, next_val) if name else next_val"},{"line_number":72,"context_line":"            if name in sorted(self.commands, key\u003dlen, reverse\u003dTrue):"},{"line_number":73,"context_line":"                cmd_ep \u003d self.commands[name]"},{"line_number":74,"context_line":"                if hasattr(cmd_ep, \u0027resolve\u0027):"},{"line_number":75,"context_line":"                    cmd_factory \u003d cmd_ep.resolve()"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_4fc372ed","line":72,"in_reply_to":"fa7ab95a_4f3372ab","updated":"2016-08-29 12:54:10.000000000","message":"Done","commit_id":"6fa9ea8d44f2ea179fd951dc69dc36dbb87ad9b0"},{"author":{"_account_id":20354,"name":"Shoham Peller","email":"shohamp@gmail.com","username":"shoham.peller"},"change_message_id":"3faf332902a6bc26ec08c8a137ff19f0b864331e","unresolved":false,"context_lines":[{"line_number":69,"context_line":"                raise ValueError(\u0027Invalid command %r\u0027 % name)"},{"line_number":70,"context_line":"            next_val \u003d search_args.pop(0)"},{"line_number":71,"context_line":"            name \u003d \u0027%s %s\u0027 % (name, next_val) if name else next_val"},{"line_number":72,"context_line":"            if name in sorted(self.commands, key\u003dlen, reverse\u003dTrue):"},{"line_number":73,"context_line":"                cmd_ep \u003d self.commands[name]"},{"line_number":74,"context_line":"                if hasattr(cmd_ep, \u0027resolve\u0027):"},{"line_number":75,"context_line":"                    cmd_factory \u003d cmd_ep.resolve()"}],"source_content_type":"text/x-python","patch_set":2,"id":"fa7ab95a_4f329260","line":72,"in_reply_to":"fa7ab95a_4f3372ab","updated":"2016-08-29 12:51:55.000000000","message":"won\u0027t the if..in only tells if the command exist? Don\u0027t you need another code to pick the right command?","commit_id":"6fa9ea8d44f2ea179fd951dc69dc36dbb87ad9b0"}]}