This is an archive of the discontinued LLVM Phabricator instance.

[libc++] Enable radability-identifier-naming for local variables and fix any problems
ClosedPublic

Authored by philnik on Feb 10 2023, 7:26 AM.

Details

Diff Detail

Event Timeline

philnik created this revision.Feb 10 2023, 7:26 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 10 2023, 7:26 AM
philnik requested review of this revision.Feb 10 2023, 7:26 AM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a reviewer: Restricted Project. · View Herald Transcript
Mordante accepted this revision.Feb 11 2023, 6:28 AM

Thanks for the cleanup, LGTM!

libcxx/include/__functional/function.h
617–618

Can you fix the indention of this line and the next lines too?

This revision is now accepted and ready to land.Feb 11 2023, 6:28 AM
This revision was landed with ongoing or failed builds.Feb 11 2023, 11:04 AM
This revision was automatically updated to reflect the committed changes.
philnik marked an inline comment as done.