Page MenuHomePhabricator

Align declarations that are preceded by different number of commas.
AbandonedPublic

Authored by nikola on Sep 9 2016, 7:07 AM.

Details

Reviewers
djasper
Summary

Having a template with more than one template argument breaks alignment of consecutive declarations. Something like this won't be correctly aligned:

int x;
std::pair<int, bool> y;

Diff Detail

Event Timeline

nikola updated this revision to Diff 70830.Sep 9 2016, 7:07 AM
nikola retitled this revision from to Align declarations that are preceded by different number of commas..
nikola updated this object.
nikola added a reviewer: djasper.
nikola added a subscriber: cfe-commits.
djasper edited edge metadata.Sep 10 2016, 1:59 AM

I think this will also be solved by https://reviews.llvm.org/D21279.

nikola abandoned this revision.Sep 11 2016, 1:48 AM

Thanks for letting me know, that patch looks more complete so I'll abandon this. I hope it lands soon!