Page MenuHomePhabricator
Feed Advanced Search

Sep 25 2018

dpayne added a comment to D51120: clang-format Additional Indent for class blocks.

Are there any comments here about this approach?

Sep 25 2018, 8:52 AM · Restricted Project

Sep 18 2018

dpayne updated the diff for D51120: clang-format Additional Indent for class blocks.

Adding the missing Style member variable.

Sep 18 2018, 6:22 PM · Restricted Project

Aug 22 2018

dpayne updated the diff for D51120: clang-format Additional Indent for class blocks.

Adding unit tests to check indents, these were mostly taken from the original patch https://reviews.llvm.org/D22505.

Aug 22 2018, 10:39 PM · Restricted Project
dpayne created D51120: clang-format Additional Indent for class blocks.
Aug 22 2018, 12:52 PM · Restricted Project

Apr 6 2018

dpayne added a comment to D43957: Fixing issue where a space was not added before a global namespace variable when SpacesInParentheses is set.

Bumping this commit. I do not have commit access so I cannot merge myself, if you have the time could you merge this commit?

Apr 6 2018, 6:27 PM

Mar 21 2018

dpayne added a comment to D43957: Fixing issue where a space was not added before a global namespace variable when SpacesInParentheses is set.

I'll make sure to do the full diff in the future. I did that for the first patch but forgot to do it in the follow up patch, I won't make that mistake in the future.

Mar 21 2018, 10:26 PM

Mar 20 2018

dpayne updated the diff for D43957: Fixing issue where a space was not added before a global namespace variable when SpacesInParentheses is set.

Adding a unit test for formatting a global namespace var with SpacesInParentheses enabled.

Mar 20 2018, 3:40 PM

Mar 1 2018

dpayne created D43957: Fixing issue where a space was not added before a global namespace variable when SpacesInParentheses is set.
Mar 1 2018, 10:18 AM