)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"064f1edb5c0fa281691dbed2232079635fa5b2ce","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3238928f_7d39358b","updated":"2022-03-18 16:21:52.000000000","message":"I\u0027m OK with doing what CPython does here.","commit_id":"ffb6ef6c4b188fae46b96a9d4528f234936b4323"}],"cinder/tests/fixtures.py":[{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"5ee224521343c3a72b2e2646fd7efdfd63df4f7c","unresolved":true,"context_lines":[{"line_number":143,"context_line":"        self.addCleanup(self._reset_warning_filters)"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"    def _reset_warning_filters(self):"},{"line_number":146,"context_line":"        warnings.filters[:] \u003d self._original_warning_filters"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"class UnHelperfulClientChannel(privsep_daemon._ClientChannel):"}],"source_content_type":"text/x-python","patch_set":3,"id":"8cc4ca63_b09c8fc8","line":146,"range":{"start_line":146,"start_character":8,"end_line":146,"end_character":27},"updated":"2022-03-08 15:46:26.000000000","message":"Is the \"[:]\" necessary?  Not sure if I\u0027m missing a detail here...","commit_id":"9e16bc5980cb370c16d1691b1e30dc1951e66a0b"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"07a9929155384d39f5d92812f4e6019a6052e3e2","unresolved":true,"context_lines":[{"line_number":143,"context_line":"        self.addCleanup(self._reset_warning_filters)"},{"line_number":144,"context_line":""},{"line_number":145,"context_line":"    def _reset_warning_filters(self):"},{"line_number":146,"context_line":"        warnings.filters[:] \u003d self._original_warning_filters"},{"line_number":147,"context_line":""},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"class UnHelperfulClientChannel(privsep_daemon._ClientChannel):"}],"source_content_type":"text/x-python","patch_set":3,"id":"4fda6269_ee705bed","line":146,"range":{"start_line":146,"start_character":8,"end_line":146,"end_character":27},"in_reply_to":"8cc4ca63_b09c8fc8","updated":"2022-03-08 16:15:34.000000000","message":"I\u0027m pretty sure there was a reason for this but I\u0027m disappointed that I never left a comment explaining this when I first wrote it (for nova). This does a different things to \"normal\" assignment, as you most likely know [1], and warnings is a global so I suspect that\u0027s the reason for it. I\u0027d need to test to be sure though.\n\nLater: aha, it\u0027s what cpython does [2]. That was likely it. I won\u0027t lie, I didn\u0027t really explore (and still haven\u0027t explored) whether this was necessary, but it also was no harm so maybe good enough (or at least follow-up territory)?\n\n[1]\n\n  \u003e\u003e\u003e a \u003d [1, 2, 3]\n  \u003e\u003e\u003e b \u003d a\n  \u003e\u003e\u003e a \u003d [0]\n  \u003e\u003e\u003e a\n  [0]\n  \u003e\u003e\u003e b\n  [1, 2, 3]\n\n  \u003e\u003e\u003e a \u003d [1, 2, 3]\n  \u003e\u003e\u003e b \u003d a\n  \u003e\u003e\u003e a[:] \u003d [0]\n  \u003e\u003e\u003e a\n  [0]\n  \u003e\u003e\u003e b\n  [0]\n\n[2] https://github.com/python/cpython/blob/5498a61c7c25db6f9e76032aa9c5153d79e09889/Lib/test/support/warnings_helper.py#L183-L189","commit_id":"9e16bc5980cb370c16d1691b1e30dc1951e66a0b"}]}