This is an archive of the discontinued LLVM Phabricator instance.

[clang-format] Expand the SpacesAroundConditions option to include catch statements
ClosedPublic

Authored by timwoj on Jan 15 2020, 10:51 AM.

Details

Event Timeline

timwoj created this revision.Jan 15 2020, 10:51 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 15 2020, 10:51 AM
Herald added a subscriber: cfe-commits. · View Herald Transcript
Eugene.Zelenko added a project: Restricted Project.
rsmmr added a subscriber: rsmmr.Jan 16 2020, 5:39 AM
This revision is now accepted and ready to land.Jan 19 2020, 4:35 AM
This revision was automatically updated to reflect the committed changes.

I know that 10.0 was branch recently. Any chance this can make it over?

I know that 10.0 was branch recently. Any chance this can make it over?

I don't personally ever do this because I only use the snapshot builds. I think if you need it in 10 then you need to ask @hans what the process is and how they decide what makes it in?

hans added a comment.Jan 21 2020, 9:56 AM

I know that 10.0 was branch recently. Any chance this can make it over?

I don't personally ever do this because I only use the snapshot builds. I think if you need it in 10 then you need to ask @hans what the process is and how they decide what makes it in?

I've cherry-picked it in 587b3b2a13d27c50006607a6b7e7458e6d7cc3fa