This is an archive of the discontinued LLVM Phabricator instance.

[clang-format] Remove double trim
ClosedPublic

Authored by HazardyKnusperkeks on Nov 23 2020, 1:29 PM.

Diff Detail

Event Timeline

Herald added a project: Restricted Project. · View Herald TranscriptNov 23 2020, 1:29 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
HazardyKnusperkeks requested review of this revision.Nov 23 2020, 1:29 PM
MyDeveloperDay accepted this revision.Nov 26 2020, 12:14 AM

Can you please clang format, but apart from that this looks fine

This revision is now accepted and ready to land.Nov 26 2020, 12:14 AM

Isn't the comment incorrect after this patch?

Isn't the comment incorrect after this patch?

I actually thought the same after I saw the diff here, but had no time yet to modify it. I will now also remove the comment.

Formatted and now superfluous comment removed.

MyDeveloperDay accepted this revision.Nov 28 2020, 9:19 AM

Isn't the comment incorrect after this patch?

This revision was automatically updated to reflect the committed changes.