)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":31083,"name":"Benjamin Schanzel","email":"benjamin.schanzel@bmw.de","username":"benjamin.schanzel"},"change_message_id":"d0b080f17009c3787ad23e19ab34ac5cb0fc3733","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"1275764b_cd5e9b76","updated":"2023-09-14 05:53:22.000000000","message":"recheck","commit_id":"9249e125a72bd05d80d9ea2f54db8914cd0cf1a0"},{"author":{"_account_id":31083,"name":"Benjamin Schanzel","email":"benjamin.schanzel@bmw.de","username":"benjamin.schanzel"},"change_message_id":"286813e5b06bab6e811c967f9b8d04ebb312950a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3651a502_003c6bc8","updated":"2023-09-18 08:22:41.000000000","message":"recheck","commit_id":"9249e125a72bd05d80d9ea2f54db8914cd0cf1a0"}],"nodepool/driver/statemachine.py":[{"author":{"_account_id":1,"name":"James E. Blair","email":"jim@acmegating.com","username":"corvus"},"change_message_id":"4f800e4eeaff6f5a2dfe1ada4e47868f3f9c7cba","unresolved":true,"context_lines":[{"line_number":253,"context_line":"                    self.start_future.result()"},{"line_number":254,"context_line":"                else:"},{"line_number":255,"context_line":"                    return"},{"line_number":256,"context_line":"            state_machine \u003d self.state_machine"},{"line_number":257,"context_line":"            if self.start_time is None:"},{"line_number":258,"context_line":"                self.start_time \u003d time.monotonic()"},{"line_number":259,"context_line":"            if (state_machine.complete and self.keyscan_future"}],"source_content_type":"text/x-python","patch_set":2,"id":"7bacd64b_3dd96380","line":256,"updated":"2023-09-25 20:22:39.000000000","message":"How about you just move this up to line 250?  Then change line 251 to drop self?  That way we can keep this method simple?","commit_id":"9249e125a72bd05d80d9ea2f54db8914cd0cf1a0"},{"author":{"_account_id":31083,"name":"Benjamin Schanzel","email":"benjamin.schanzel@bmw.de","username":"benjamin.schanzel"},"change_message_id":"9df90d19628463f233b60ded3cd329473e9b3867","unresolved":true,"context_lines":[{"line_number":253,"context_line":"                    self.start_future.result()"},{"line_number":254,"context_line":"                else:"},{"line_number":255,"context_line":"                    return"},{"line_number":256,"context_line":"            state_machine \u003d self.state_machine"},{"line_number":257,"context_line":"            if self.start_time is None:"},{"line_number":258,"context_line":"                self.start_time \u003d time.monotonic()"},{"line_number":259,"context_line":"            if (state_machine.complete and self.keyscan_future"}],"source_content_type":"text/x-python","patch_set":2,"id":"20bd836e_9298361b","line":256,"in_reply_to":"7bacd64b_3dd96380","updated":"2023-09-26 12:44:15.000000000","message":"That\u0027s what I had in PS1 of this change. But self.state_machine might be None at line 250 and I think a later assignment of self.state_machine would break the reference we create here.","commit_id":"9249e125a72bd05d80d9ea2f54db8914cd0cf1a0"}]}