#include "UnwrappedLineParser.h"
#include "FormatToken.h"
+#include "clang/Basic/TokenKinds.h"
#include "llvm/ADT/STLExtras.h"
#include "llvm/Support/Debug.h"
#include "llvm/Support/raw_ostream.h"
Keywords.kw_import, tok::kw_export);
}
-// Checks whether a token is a type in K&R C (aka C78).
-static bool isC78Type(const FormatToken &Tok) {
- return Tok.isOneOf(tok::kw_char, tok::kw_short, tok::kw_int, tok::kw_long,
- tok::kw_unsigned, tok::kw_float, tok::kw_double,
- tok::identifier);
-}
-
// This function checks whether a token starts the first parameter declaration
// in a K&R C (aka C78) function definition, e.g.:
// int f(a, b)
if (!Tok)
return false;
- if (!isC78Type(*Tok) &&
- !Tok->isOneOf(tok::kw_register, tok::kw_struct, tok::kw_union))
+ if (!Tok->isOneOf(tok::kw_int, tok::kw_char, tok::kw_float, tok::kw_double,
+ tok::kw_struct, tok::kw_union, tok::kw_long, tok::kw_short,
+ tok::kw_unsigned, tok::kw_register))
return false;
Tok = Tok->Previous;
case tok::r_brace:
addUnwrappedLine();
return;
- case tok::l_paren: {
+ case tok::l_paren:
parseParens();
// Break the unwrapped line if a K&R C function definition has a parameter
// declaration.
break;
if (!Previous || Previous->isNot(tok::identifier))
break;
- const FormatToken *PrevPrev = Previous->Previous;
- if (!PrevPrev || (!isC78Type(*PrevPrev) && PrevPrev->isNot(tok::star)))
+ if (Previous->Previous && Previous->Previous->is(tok::at))
break;
- const FormatToken *Next = AllTokens[Tokens->getPosition() + 1];
- if (Next && Next->isOneOf(tok::l_paren, tok::semi))
- break;
- if (isC78ParameterDecl(FormatTok)) {
+ if (!Line->Tokens.begin()->Tok->is(tok::kw_typedef) &&
+ isC78ParameterDecl(FormatTok)) {
addUnwrappedLine();
return;
}
break;
- }
case tok::kw_operator:
nextToken();
if (FormatTok->isBinaryOperator())
" return a + b < c;\n"
"};",
Style);
- verifyFormat("byte *\n" // Break here.
- "f(a)\n" // Break here.
- "byte a[];\n"
- "{\n"
- " return a;\n"
- "}",
- Style);
- verifyFormat("bool f(int a, int) override;\n"
- "Bar g(int a, Bar) final; // comment",
- Style);
// The return breaking style doesn't affect:
// * function and object definitions with attribute-like macros