[libcxx-commits] [PATCH] D108263: [libc++] Update and alphabetize CREDITS.TXT

[Kent Ross via Phabricator via libcxx-commits]

mumbleskates marked an inline comment as done.
mumbleskates added inline comments.


================
Comment at: libcxx/CREDITS.TXT:134-136
+N: Ruben Van Boxem
+E: vanboxem dot ruben at gmail dot com
+D: Initial Windows patches.
----------------
Quuxplusone wrote:
> FWIW, I assume this name was originally alphabetized under "B" because that's the local collation order for surnames: Ruben van BOXEM; compare Klass de VRIES below. (But then Michael van der WESTHUIZEN would be wrong?)
> If we're alphabetizing by surname, not just `(cdr $name)`, then ZHANG Xiongpang remains in the wrong place.
> Names are politics and politics are impossible to get right. Personally I doubt that this entire effort is helpful at all. But it's also pretty harmless, so whatever.
Happy also to revert this specific one and fall back to the self-indication of where the surname begins (at 'B' for Ruben and at at 'v' for Michael). I don't think it's probably a huge deal as this is probably the single least important and annoying appearance of one's last name.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D108263/new/

https://reviews.llvm.org/D108263