)]}'
{"neutron/api/v2/base.py":[{"author":{"_account_id":7448,"name":"Carl Baldwin","email":"carl@ecbaldwin.net","username":"carl-baldwin"},"change_message_id":"c99a9f96a07467623542665e5554c4af3dad8c17","unresolved":false,"context_lines":[{"line_number":195,"context_line":"                    resource \u003d self._item(request,"},{"line_number":196,"context_line":"                                          id,"},{"line_number":197,"context_line":"                                          do_authz\u003dTrue,"},{"line_number":198,"context_line":"                                          field_list\u003dNone,"},{"line_number":199,"context_line":"                                          parent_id\u003dparent_id)"},{"line_number":200,"context_line":"                except oslo_policy.PolicyNotAuthorized:"},{"line_number":201,"context_line":"                    msg \u003d _(\u0027The resource could not be found.\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a41bdd9_ff61d2d3","line":198,"updated":"2015-07-15 17:03:44.000000000","message":"The commit message didn\u0027t mention adding field_list to the list of explicit args.  It isn\u0027t necessary.  Is there a reason that you add it here rather than accept the default of None in the method definition?","commit_id":"87c3ea7e739b1d8af61e5fae39e1d86aa53bc1b2"},{"author":{"_account_id":12412,"name":"Shang Yong","email":"yong.shang@me.com","username":"bshang"},"change_message_id":"7da539641de4e9f99d80e8512276288c66fe6224","unresolved":false,"context_lines":[{"line_number":195,"context_line":"                    resource \u003d self._item(request,"},{"line_number":196,"context_line":"                                          id,"},{"line_number":197,"context_line":"                                          do_authz\u003dTrue,"},{"line_number":198,"context_line":"                                          field_list\u003dNone,"},{"line_number":199,"context_line":"                                          parent_id\u003dparent_id)"},{"line_number":200,"context_line":"                except oslo_policy.PolicyNotAuthorized:"},{"line_number":201,"context_line":"                    msg \u003d _(\u0027The resource could not be found.\u0027)"}],"source_content_type":"text/x-python","patch_set":3,"id":"9a41bdd9_9af078cb","line":198,"in_reply_to":"9a41bdd9_ff61d2d3","updated":"2015-07-17 08:29:21.000000000","message":"Yes, it\u0027s not necessary, but it is more clear here to pass all three optional arguments with values.","commit_id":"87c3ea7e739b1d8af61e5fae39e1d86aa53bc1b2"}]}