)]}'
{"cliff/interactive.py":[{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"ed4a00a8e53afdacead7cb45c426a5652674015f","unresolved":false,"context_lines":[{"line_number":115,"context_line":"                # NOTE(coreycb): This try path can be removed once"},{"line_number":116,"context_line":"                # requirements.txt has cmd2 \u003e\u003d 0.7.3."},{"line_number":117,"context_line":"                self.default(self.parsed(\u0027help \u0027 + arg))"},{"line_number":118,"context_line":"            except AttributeError:"},{"line_number":119,"context_line":"                self.default(self.parser_manager.parsed(\u0027help \u0027 + arg))"},{"line_number":120,"context_line":"        else:"},{"line_number":121,"context_line":"            cmd2.Cmd.do_help(self, arg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"ff6b8bd7_74cbcfa4","line":118,"updated":"2018-03-06 15:07:43.000000000","message":"Hmm, I suppose we don\u0027t expect self.default() or self.parsed() to raise AttributeError from any action they take?\n\nWould it be safer to save self.parsed or self.parser_manager.parsed to a variable within the try block, so we know that the AttributeError is because self.parsed isn\u0027t there? Then we could call self.default(parse_func(\u0027help \u0027 + arg)) outside of the try/except.","commit_id":"78f9eca1051c2a580f89c9d0e0a9e0234632877a"},{"author":{"_account_id":11805,"name":"Corey Bryant","email":"corey.bryant@canonical.com","username":"coreycb"},"change_message_id":"0b1d0f318a4b71b4893ade4b885f9ebd6daa6eb2","unresolved":false,"context_lines":[{"line_number":115,"context_line":"                # NOTE(coreycb): This try path can be removed once"},{"line_number":116,"context_line":"                # requirements.txt has cmd2 \u003e\u003d 0.7.3."},{"line_number":117,"context_line":"                self.default(self.parsed(\u0027help \u0027 + arg))"},{"line_number":118,"context_line":"            except AttributeError:"},{"line_number":119,"context_line":"                self.default(self.parser_manager.parsed(\u0027help \u0027 + arg))"},{"line_number":120,"context_line":"        else:"},{"line_number":121,"context_line":"            cmd2.Cmd.do_help(self, arg)"}],"source_content_type":"text/x-python","patch_set":4,"id":"ff6b8bd7_54700b73","line":118,"in_reply_to":"ff6b8bd7_74cbcfa4","updated":"2018-03-06 15:10:23.000000000","message":"That would make more sense, yes. Let me resubmit.","commit_id":"78f9eca1051c2a580f89c9d0e0a9e0234632877a"}]}