As the reviewer of the patch in question, yes I was exhausted by multiple rounds of review restating the same feedback. However, in this situation, the actual change is minimal and trivially correct. Most of the review concerned the test changes. In this case most of the contention was related to retaining test coverage of pre-existing breakage. I think the ultimate change is correct, but I’m not sure what the correct solution is for tests that are known to fail a sanitizer but are there to track that the test fails with a sanitizer.