I made it longer so the two lines (__attribute__((foo)) vs. [[foo]]) had the same length.
If I shorten it much more, everything goes on one line. On Wed, Mar 7, 2018, 03:42 Krasimir Georgiev via Phabricator < revi...@reviews.llvm.org> wrote: > krasimir added inline comments. > > > ================ > Comment at: unittests/Format/FormatTest.cpp:6068-6069 > + verifyFormat("void f() [[deprecated(\"so sorry\")]];"); > + verifyFormat("aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa\n" > + " [[unused]] aaaaaaaaaaaaaaaaaaaaaaa(int i);"); > +} > ---------------- > benhamilton wrote: > > I couldn't figure out why this breaks before the `[` in `[[unused]]`, > when the identical test using `__attribute__((unused))` above does *not* > break before the `__attribute__`. > > > > I ran with `-debug` and the two seem fairly similar. Can anyone help > shed light on this? > > > > ## With `__attribute__((unused))` > > > > P8067 > > > > ## With `[[unused]]` > > > > P8068 > The block of `a`-s appears to be longer here than in the `__attribute__` > example. That might be a reason. > > > Repository: > rC Clang > > https://reviews.llvm.org/D43902 > > > >
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits