)]}'
{"cliff/command.py":[{"author":{"_account_id":20150,"name":"Qichen Hao","email":"adsl123gg2008@gmail.com","username":"adsl123gg2008"},"change_message_id":"8e503dd3d5bbe049d85c819a07fa84f0e47f9c4c","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        method to wrap :meth:`take_action`."},{"line_number":53,"context_line":"        \"\"\""},{"line_number":54,"context_line":"        result \u003d self.take_action(parsed_args)"},{"line_number":55,"context_line":"        return result if result is not None else 0"}],"source_content_type":"text/x-python","patch_set":3,"id":"1af94dfe_66b84de6","line":55,"range":{"start_line":55,"start_character":14,"end_line":55,"end_character":50},"updated":"2016-03-17 10:22:43.000000000","message":"I think we need the return value,in some case the user run the command and want to wheather the command is successly executed.","commit_id":"f1504dc814fe4ef740ca11ee21f060f14a3d03e3"},{"author":{"_account_id":2472,"name":"Doug Hellmann","email":"dhellmann@redhat.com","username":"doug-hellmann"},"change_message_id":"2a33e727486c58e65b3e9eaca190503a341aec0e","unresolved":false,"context_lines":[{"line_number":52,"context_line":"        method to wrap :meth:`take_action`."},{"line_number":53,"context_line":"        \"\"\""},{"line_number":54,"context_line":"        result \u003d self.take_action(parsed_args)"},{"line_number":55,"context_line":"        return result if result is not None else 0"}],"source_content_type":"text/x-python","patch_set":3,"id":"3a7e1126_af3e75d8","line":55,"updated":"2015-12-22 13:40:36.000000000","message":"This potentially allows result to be any type, including arbitrary things like lists and objects. The calling code expects it to always be an integer. Something like this should ensure that the value is always cast to an integer:\n\n  return int(bool(result))","commit_id":"f1504dc814fe4ef740ca11ee21f060f14a3d03e3"}]}