)]}'
{"nova/tests/fake_network.py":[{"author":{"_account_id":642,"name":"Matt Dietz","email":"matthew.dietz@gmail.com","username":"cerberus"},"change_message_id":"c5583213ba2fc2d88878b146a4febea364c6e0d9","unresolved":false,"context_lines":[{"line_number":55,"context_line":"        self.__setattr__(key, val)"},{"line_number":56,"context_line":""},{"line_number":57,"context_line":"    def get_config(self, instance, network, mapping, image_meta,"},{"line_number":58,"context_line":"                   extra_specs\u003d{}):"},{"line_number":59,"context_line":"        conf \u003d libvirt_config.LibvirtConfigGuestInterface()"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"        for attr, val in conf.__dict__.iteritems():"}],"source_content_type":"text/x-python","patch_set":4,"id":"AAAAN3%2F%2F6og%3D","line":58,"updated":"2013-03-06 16:21:28.000000000","message":"Generally a bad idea to set a default value to any sort of object, as Python binds the function args to those objects by reference. Any changes within the scope of the function would be appended to the same object over time.\n\nI\u0027d prefer you change it, as someone mucking with tests later could create some really confusing situations.","commit_id":"f13257b311a34e073ff41f2e5d33e3477a4e5d8c"}]}