)]}'
{"tools/conf/extract_opts.py":[{"author":{"_account_id":2243,"name":"John Griffith","email":"john.griffith8@gmail.com","username":"john-griffith"},"change_message_id":"32c2406af6a25df2413bb4c779a45df88b145da4","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        mod_obj \u003d importutils.import_module(mod_str)"},{"line_number":82,"context_line":"    except Exception, e:"},{"line_number":83,"context_line":"        sys.stderr.write(\"Failed to collect options from module %s: %s\\n\" % ("},{"line_number":84,"context_line":"            mod_str, str(e)))"},{"line_number":85,"context_line":"        return"},{"line_number":86,"context_line":"    _list_opts(mod_obj, mod_str)"},{"line_number":87,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2FfTU%3D","line":84,"updated":"2013-06-01 01:55:38.000000000","message":"Just curious, why not simply add logging output to the exisitng exceptions, rather than throw them out and just use a generic?","commit_id":"27e261376fc03655a5f90603078a3a8623ddf32f"},{"author":{"_account_id":7706,"name":"Anton Frolov","email":"frolov.anton@gmail.com","username":"anton"},"change_message_id":"0c45423457476b5433dbb2950eb09b605b7e734b","unresolved":false,"context_lines":[{"line_number":81,"context_line":"        mod_obj \u003d importutils.import_module(mod_str)"},{"line_number":82,"context_line":"    except Exception, e:"},{"line_number":83,"context_line":"        sys.stderr.write(\"Failed to collect options from module %s: %s\\n\" % ("},{"line_number":84,"context_line":"            mod_str, str(e)))"},{"line_number":85,"context_line":"        return"},{"line_number":86,"context_line":"    _list_opts(mod_obj, mod_str)"},{"line_number":87,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAPH%2F%2Feq4%3D","line":84,"in_reply_to":"AAAAPH%2F%2FfTU%3D","updated":"2013-06-02 12:15:49.000000000","message":"Old version already catch all exceptions derived from Exception class. Also exception handling code will be the same for less generic exceptions, so I see no point keeping it.","commit_id":"27e261376fc03655a5f90603078a3a8623ddf32f"}]}
