This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] Fixed a number of typos
ClosedPublic

Authored by GabrielRavier on Aug 1 2022, 7:55 AM.

Details

Reviewers
philnik
Group Reviewers
Restricted Project
Commits
rGc23e2c015fb7: [libcxx] Fixed a number of typos
Summary

I went over the output of the following mess of a command:

(ulimit -m 2000000; ulimit -v 2000000; git ls-files -z | parallel --xargs -0 cat | aspell list --mode=none --ignore-case | grep -E '^[A-Za-z][a-z]*$' | sort | uniq -c | sort -n | grep -vE '.{25}' | aspell pipe -W3 | grep : | cut -d' ' -f2 | less)

and proceeded to spend a few days looking at it to find probable typos
and fixed a few hundred of them in all of the llvm project (note, the
ones I found are not anywhere near all of them, but it seems like a
good start).

Diff Detail

Event Timeline

GabrielRavier created this revision.Aug 1 2022, 7:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 1 2022, 7:55 AM
GabrielRavier requested review of this revision.Aug 1 2022, 7:55 AM
Herald added a project: Restricted Project. · View Herald TranscriptAug 1 2022, 7:55 AM
Herald added a reviewer: Restricted Project. · View Herald Transcript
philnik accepted this revision.Aug 1 2022, 8:00 AM
philnik added a subscriber: philnik.

Thanks! Do you need someone else to commit it? In that case, please provide Your Name <your@email.address>.

This revision is now accepted and ready to land.Aug 1 2022, 8:00 AM
GabrielRavier added a comment.EditedAug 1 2022, 8:12 AM

Well the commit on clang-tools-extra went in fine with my correct username/email address pair, so I assume it'd be possible for you to get that information elsewhere (shouldn't Phabricator provide it ?), but it should be Gabriel Ravier <gabravier@gmail.com>.

This revision was automatically updated to reflect the committed changes.

Well the commit on clang-tools-extra went in fine with my correct username/email address pair, so I assume it'd be possible for you to get that information elsewhere (shouldn't Phabricator provide it ?), but it should be Gabriel Ravier <gabravier@gmail.com>.

Yes, it's part of the patch, but I think it's better to ask than to accidentally commit with the wrong information.