[BTree][NFC] Rephrase some comments#427
Conversation
|
@swift-ci test |
|
@lorentey I believe FWIW… there might also be examples of outdated language that can't be deleted easily like comments… there could be constant strings passed to logger or even symbols like function names that contain the outdated language. You might also for this specific diff think about leaving these comments in but just migrating the language to something modern and inclusive as a guide for future engineers to migrate future symbols. cc @Serena748 |
|
Yep. |
|
C.f. #426 |
|
I'm not particularly fond of the idea that assertions need to explicitly explain that they're for checking invariants. (That is what assertions are for.) Therefore, the right fix was to remove these. Please file PRs or issues if you find any similar cases. These were found by an automated tool; I find it encouraging that the only hits were in experimental code that hasn't yet gone through the usual implementation/style review. |
Change some unfortunate wording around invariant violations.
Checklist