)]}'
{"designate/objects/base.py":[{"author":{"_account_id":15171,"name":"Eric Larson","email":"eric.larson@rackspace.com","username":"elarson"},"change_message_id":"86531ff9c1a94dfd60b60d47eb8b343e9fb9f69a","unresolved":false,"context_lines":[{"line_number":474,"context_line":"        instance._obj_list_additions \u003d set()"},{"line_number":475,"context_line":"        instance._obj_list_removals \u003d set("},{"line_number":476,"context_line":"            [DesignateObject.from_primitive(v) for v in"},{"line_number":477,"context_line":"             primitive[\u0027designate_object.list_removals\u0027]])"},{"line_number":478,"context_line":""},{"line_number":479,"context_line":"        for field, value in primitive[\u0027designate_object.data\u0027].items():"},{"line_number":480,"context_line":"            if field \u003d\u003d \u0027objects\u0027:"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa1b9901_b6189d1f","line":477,"updated":"2015-08-19 19:48:24.000000000","message":"A little nitpicky but I\u0027d format the code with:\n\n  set([\n    DesignateObject.from_primitive(v) for v in \n    primitive[\u0027designate_object.list_removals\u0027]\n  ])\n\nThe double square bracket is a little confusing and this may also help with diffs in the future.","commit_id":"bcb0c144c138d6a7e5f017a4d2bba947f6fc75b7"},{"author":{"_account_id":741,"name":"Kiall Mac Innes","email":"kiall@macinnes.ie","username":"kiall"},"change_message_id":"abd9ee628d98fb2f3da5beb9a6a1778093d5b1dc","unresolved":false,"context_lines":[{"line_number":474,"context_line":"        instance._obj_list_additions \u003d set()"},{"line_number":475,"context_line":"        instance._obj_list_removals \u003d set("},{"line_number":476,"context_line":"            [DesignateObject.from_primitive(v) for v in"},{"line_number":477,"context_line":"             primitive[\u0027designate_object.list_removals\u0027]])"},{"line_number":478,"context_line":""},{"line_number":479,"context_line":"        for field, value in primitive[\u0027designate_object.data\u0027].items():"},{"line_number":480,"context_line":"            if field \u003d\u003d \u0027objects\u0027:"}],"source_content_type":"text/x-python","patch_set":4,"id":"fa1b9901_bea2b3fb","line":477,"in_reply_to":"fa1b9901_b6189d1f","updated":"2015-08-19 20:45:34.000000000","message":"Done","commit_id":"bcb0c144c138d6a7e5f017a4d2bba947f6fc75b7"},{"author":{"_account_id":16962,"name":"Federico Ceratto","email":"federico.ceratto@gmail.com","username":"ceratto"},"change_message_id":"a646509174e9191fd7f00382f6d342c178a68365","unresolved":false,"context_lines":[{"line_number":560,"context_line":"            \u0027designate_object.changes\u0027: list(self._obj_changes),"},{"line_number":561,"context_line":"            \u0027designate_object.original_values\u0027: dict("},{"line_number":562,"context_line":"                self._obj_original_values),"},{"line_number":563,"context_line":"            \u0027designate_object.list_additions\u0027: ["},{"line_number":564,"context_line":"                self.objects.index(o) for o in self._obj_list_additions"},{"line_number":565,"context_line":"            ],"},{"line_number":566,"context_line":"            \u0027designate_object.list_removals\u0027: ["}],"source_content_type":"text/x-python","patch_set":6,"id":"9a1a9d01_7cc1ece3","line":563,"updated":"2015-09-25 11:06:13.000000000","message":"Style nitpick: are we ok with doing method calls and list comprehensions nested inside the dictionary creation inside a return statement?","commit_id":"cab4d39369bfe39c6ecc033482a3ea3987b26496"}]}