)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":1653,"name":"garyk","email":"gkotton@vmware.com","username":"garyk"},"unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Bug #1086232"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"We introduce the action\u003dclear to unkown agument part of"},{"line_number":12,"context_line":"the command line. With this option, user can specify which"},{"line_number":13,"context_line":"attribut will have None value."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"AAAAMX%2F%2F1oU%3D","line":11,"updated":"2012-12-13 07:55:44.000000000","message":"agument \u003d\u003e argument","commit_id":"6cee75e7663930fa92271a74086dd5274721ebe3"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Bug #1086232"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"We introduce the action\u003dclear to unkown argument part of"},{"line_number":12,"context_line":"the command line. With this option, user can specify which"},{"line_number":13,"context_line":"attribut will have None value."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAPn%2F%2F1cU%3D","line":11,"updated":"2013-06-20 05:56:56.000000000","message":"unknown","commit_id":"33725f52c8bfb42755e5e2297d800655d38e037d"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"46985fad0ba998b1b81d8b2f4d5a67eb4ec6bc4d","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Bug #1086232"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"We introduce the action\u003dclear to unkown argument part of"},{"line_number":12,"context_line":"the command line. With this option, user can specify which"},{"line_number":13,"context_line":"attribut will have None value."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAPn%2F%2FgIQ%3D","line":11,"in_reply_to":"AAAAPn%2F%2F1cU%3D","updated":"2013-06-29 00:41:40.000000000","message":"Done","commit_id":"33725f52c8bfb42755e5e2297d800655d38e037d"},{"author":{"_account_id":748,"name":"Armando Migliaccio","email":"armamig@gmail.com","username":"armando-migliaccio"},"unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"We introduce the action\u003dclear to unkown argument part of"},{"line_number":12,"context_line":"the command line. With this option, user can specify which"},{"line_number":13,"context_line":"attribut will have None value."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"For example to set dns_nameservers of subnet to None, we use:"},{"line_number":16,"context_line":"quantum subnet-update subnetname --dns_nameservers action\u003dclear"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAPn%2F%2F1cI%3D","line":13,"updated":"2013-06-20 05:56:56.000000000","message":"attribute","commit_id":"33725f52c8bfb42755e5e2297d800655d38e037d"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"46985fad0ba998b1b81d8b2f4d5a67eb4ec6bc4d","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"We introduce the action\u003dclear to unkown argument part of"},{"line_number":12,"context_line":"the command line. With this option, user can specify which"},{"line_number":13,"context_line":"attribut will have None value."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"For example to set dns_nameservers of subnet to None, we use:"},{"line_number":16,"context_line":"quantum subnet-update subnetname --dns_nameservers action\u003dclear"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAAPn%2F%2FgIM%3D","line":13,"in_reply_to":"AAAAPn%2F%2F1cI%3D","updated":"2013-06-29 00:41:40.000000000","message":"Done","commit_id":"33725f52c8bfb42755e5e2297d800655d38e037d"}],"quantumclient/quantum/v2_0/__init__.py":[{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"3c145d166ef50a6f68bcd0f482ee06cf100963a8","unresolved":false,"context_lines":[{"line_number":102,"context_line":"            current_arg.update({\u0027nargs\u0027: \u0027+\u0027})"},{"line_number":103,"context_line":"        elif _value_number \u003d\u003d 0:"},{"line_number":104,"context_line":"            # Pop the argument for which there is no values specified"},{"line_number":105,"context_line":"            # so that we will allow default value used in parser"},{"line_number":106,"context_line":"            if _list_flag or current_type_str \u003d\u003d \u0027list\u0027:"},{"line_number":107,"context_line":"                current_arg.update({\u0027nargs\u0027: \u0027*\u0027})"},{"line_number":108,"context_line":"            _values_specs.pop()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F6mg%3D","line":105,"updated":"2012-12-07 09:20:01.000000000","message":"We have removed current_arg.update({\u0027action\u0027:\u0027store_true\u0027})\n\nIs that intentional? I do not recall any value spec where we use the flag only, but there could be in the future.","commit_id":"59cf53c48c1b88b29943745587457fd1c9603e6d"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"46985fad0ba998b1b81d8b2f4d5a67eb4ec6bc4d","unresolved":false,"context_lines":[{"line_number":102,"context_line":"            current_arg.update({\u0027nargs\u0027: \u0027+\u0027})"},{"line_number":103,"context_line":"        elif _value_number \u003d\u003d 0:"},{"line_number":104,"context_line":"            # Pop the argument for which there is no values specified"},{"line_number":105,"context_line":"            # so that we will allow default value used in parser"},{"line_number":106,"context_line":"            if _list_flag or current_type_str \u003d\u003d \u0027list\u0027:"},{"line_number":107,"context_line":"                current_arg.update({\u0027nargs\u0027: \u0027*\u0027})"},{"line_number":108,"context_line":"            _values_specs.pop()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F6Aw%3D","line":105,"in_reply_to":"AAAAMX%2F%2F6cs%3D","updated":"2012-12-07 23:51:19.000000000","message":"Yes, it has.","commit_id":"59cf53c48c1b88b29943745587457fd1c9603e6d"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"31deede021099f277b755591cedd75d163652a6b","unresolved":false,"context_lines":[{"line_number":102,"context_line":"            current_arg.update({\u0027nargs\u0027: \u0027+\u0027})"},{"line_number":103,"context_line":"        elif _value_number \u003d\u003d 0:"},{"line_number":104,"context_line":"            # Pop the argument for which there is no values specified"},{"line_number":105,"context_line":"            # so that we will allow default value used in parser"},{"line_number":106,"context_line":"            if _list_flag or current_type_str \u003d\u003d \u0027list\u0027:"},{"line_number":107,"context_line":"                current_arg.update({\u0027nargs\u0027: \u0027*\u0027})"},{"line_number":108,"context_line":"            _values_specs.pop()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F6cs%3D","line":105,"in_reply_to":"AAAAMX%2F%2F6fM%3D","updated":"2012-12-07 14:50:00.000000000","message":"Wouldn\u0027t that make the CLI backward incompatible?","commit_id":"59cf53c48c1b88b29943745587457fd1c9603e6d"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"85b9fe4de522226d25f20b73063fa80d9ed1b9d7","unresolved":false,"context_lines":[{"line_number":102,"context_line":"            current_arg.update({\u0027nargs\u0027: \u0027+\u0027})"},{"line_number":103,"context_line":"        elif _value_number \u003d\u003d 0:"},{"line_number":104,"context_line":"            # Pop the argument for which there is no values specified"},{"line_number":105,"context_line":"            # so that we will allow default value used in parser"},{"line_number":106,"context_line":"            if _list_flag or current_type_str \u003d\u003d \u0027list\u0027:"},{"line_number":107,"context_line":"                current_arg.update({\u0027nargs\u0027: \u0027*\u0027})"},{"line_number":108,"context_line":"            _values_specs.pop()"}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F6fM%3D","line":105,"in_reply_to":"AAAAMX%2F%2F6mg%3D","updated":"2012-12-07 13:56:53.000000000","message":"Previously, if we don\u0027t give the value for a argument, it will be regarded as bool argument and the value will be True.\n\nNow I want to give the value None to all the arguments without value specified.","commit_id":"59cf53c48c1b88b29943745587457fd1c9603e6d"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"change_message_id":"3c145d166ef50a6f68bcd0f482ee06cf100963a8","unresolved":false,"context_lines":[{"line_number":137,"context_line":"    for _item in values_specs:"},{"line_number":138,"context_line":"        if _item.startswith(\u0027--\u0027):"},{"line_number":139,"context_line":"            # Deal with previous argument if any"},{"line_number":140,"context_line":"            _deal_with_last_argument("},{"line_number":141,"context_line":"                current_arg, _value_number, current_type_str,"},{"line_number":142,"context_line":"                _list_flag, _values_specs)"},{"line_number":143,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F6mo%3D","line":140,"updated":"2012-12-07 09:20:01.000000000","message":"Then the method should probably be named \"handle_ previous_argument\" [choice is yours - this is just a suggestion]","commit_id":"59cf53c48c1b88b29943745587457fd1c9603e6d"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"85b9fe4de522226d25f20b73063fa80d9ed1b9d7","unresolved":false,"context_lines":[{"line_number":137,"context_line":"    for _item in values_specs:"},{"line_number":138,"context_line":"        if _item.startswith(\u0027--\u0027):"},{"line_number":139,"context_line":"            # Deal with previous argument if any"},{"line_number":140,"context_line":"            _deal_with_last_argument("},{"line_number":141,"context_line":"                current_arg, _value_number, current_type_str,"},{"line_number":142,"context_line":"                _list_flag, _values_specs)"},{"line_number":143,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"AAAAMX%2F%2F6fg%3D","line":140,"in_reply_to":"AAAAMX%2F%2F6mo%3D","updated":"2012-12-07 13:56:53.000000000","message":"Done","commit_id":"59cf53c48c1b88b29943745587457fd1c9603e6d"},{"author":{"_account_id":261,"name":"Salvatore Orlando","email":"salv.orlando@gmail.com","username":"salvatore-orlando"},"unresolved":false,"context_lines":[{"line_number":181,"context_line":"        elif _item \u003d\u003d \u0027action\u003dclear\u0027:"},{"line_number":182,"context_line":"            _clear_flag \u003d True"},{"line_number":183,"context_line":"            continue"},{"line_number":184,"context_line":"        if not _item.startswith(\u0027--\u0027):"},{"line_number":185,"context_line":"            # Make sure -- occurs first"},{"line_number":186,"context_line":"            if not current_item or \u0027\u003d\u0027 in current_item:"},{"line_number":187,"context_line":"                raise exceptions.CommandError("}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAM3%2F%2F%2Fhs%3D","line":184,"updated":"2012-12-17 14:04:15.000000000","message":"I can follow the logic here, but I was wondering that probably just an else instead of a new if block would have had the same effects, with a better readability.","commit_id":"33725f52c8bfb42755e5e2297d800655d38e037d"},{"author":{"_account_id":2874,"name":"yong sheng gong","email":"gong.yongsheng@99cloud.net","username":"gongysh"},"change_message_id":"46985fad0ba998b1b81d8b2f4d5a67eb4ec6bc4d","unresolved":false,"context_lines":[{"line_number":181,"context_line":"        elif _item \u003d\u003d \u0027action\u003dclear\u0027:"},{"line_number":182,"context_line":"            _clear_flag \u003d True"},{"line_number":183,"context_line":"            continue"},{"line_number":184,"context_line":"        if not _item.startswith(\u0027--\u0027):"},{"line_number":185,"context_line":"            # Make sure -- occurs first"},{"line_number":186,"context_line":"            if not current_item or \u0027\u003d\u0027 in current_item:"},{"line_number":187,"context_line":"                raise exceptions.CommandError("}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAPn%2F%2FgII%3D","line":184,"in_reply_to":"AAAAM3%2F%2F%2Fhs%3D","updated":"2013-06-29 00:41:40.000000000","message":"Done","commit_id":"33725f52c8bfb42755e5e2297d800655d38e037d"}],"quantumclient/tests/unit/test_casual_args.py":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"unresolved":false,"context_lines":[{"line_number":71,"context_line":"        self.assertEqual(\u0027value1\u0027, arg1[0][\u0027key1\u0027])"},{"line_number":72,"context_line":"        self.assertEqual(\u0027value2\u0027, arg1[0][\u0027key2\u0027])"},{"line_number":73,"context_line":""},{"line_number":74,"context_line":"    def test_defalt_values(self):"},{"line_number":75,"context_line":"        _specs \u003d [\u0027--anyarg\u0027, \u0027action\u003dclear\u0027]"},{"line_number":76,"context_line":"        args \u003d quantumV20.parse_args_to_dict(_specs)"},{"line_number":77,"context_line":"        self.assertEqual(None, args[\u0027anyarg\u0027])"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMX%2F%2F4Oc%3D","line":74,"updated":"2012-12-11 08:22:47.000000000","message":"typo: defalt -\u003e default?\n\ntest_clear_value may be another choice of the test name.","commit_id":"a9d8526d0bbcd509d49df502ef874852694ced09"}]}