1
0
Fork 0
mirror of https://github.com/beefytech/Beef.git synced 2025-06-08 19:48:20 +02:00

Fixed missing-comma formatting

This commit is contained in:
Brian Fiete 2020-09-06 05:28:19 -07:00
parent d11b09dd07
commit 0eea8eeef6
2 changed files with 33 additions and 26 deletions

View file

@ -400,6 +400,13 @@ public:
nodeRef->Accept(this); nodeRef->Accept(this);
mCurChildRef = NULL; mCurChildRef = NULL;
} }
template <typename T>
void VisitChildNoRef(const T& nodeRef)
{
if ((BfAstNode*)nodeRef == NULL)
return;
nodeRef->Accept(this);
};
public: public:
BfStructuralVisitor(); BfStructuralVisitor();

View file

@ -868,7 +868,7 @@ void BfPrinter::Visit(BfAttributedExpression* attribExpr)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(indexerExpr->mCommas[i - 1]); VisitChildNoRef(indexerExpr->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(indexerExpr->mArguments[i]); VisitChild(indexerExpr->mArguments[i]);
@ -972,7 +972,7 @@ void BfPrinter::Visit(BfAttributeDirective* attributeDirective)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(attributeDirective->mCommas[i - 1]); VisitChildNoRef(attributeDirective->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(attributeDirective->mArguments[i]); VisitChild(attributeDirective->mArguments[i]);
@ -992,7 +992,7 @@ void BfPrinter::Visit(BfGenericParamsDeclaration* genericParams)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(genericParams->mCommas[i - 1]); VisitChildNoRef(genericParams->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(genericParams->mGenericParams[i]); VisitChild(genericParams->mGenericParams[i]);
@ -1033,7 +1033,7 @@ void BfPrinter::Visit(BfGenericConstraintsDeclaration* genericConstraints)
if (i > 0) if (i > 0)
{ {
if (!genericConstraint->mCommas.IsEmpty()) if (!genericConstraint->mCommas.IsEmpty())
VisitChild(genericConstraint->mCommas[i - 1]); VisitChildNoRef(genericConstraint->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(genericConstraint->mConstraintTypes[i]); VisitChild(genericConstraint->mConstraintTypes[i]);
@ -1050,7 +1050,7 @@ void BfPrinter::Visit(BfGenericArgumentsNode* genericArgumentsNode)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(genericArgumentsNode->mCommas[i - 1]); VisitChildNoRef(genericArgumentsNode->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(genericArgumentsNode->mGenericArgs[i]); VisitChild(genericArgumentsNode->mGenericArgs[i]);
@ -1217,7 +1217,7 @@ void BfPrinter::Visit(BfInitializerExpression* initExpr)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(initExpr->mCommas[i - 1]); VisitChildNoRef(initExpr->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(initExpr->mValues[i]); VisitChild(initExpr->mValues[i]);
@ -1234,7 +1234,7 @@ void BfPrinter::Visit(BfCollectionInitializerExpression* initExpr)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(initExpr->mCommas[i - 1]); VisitChildNoRef(initExpr->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(initExpr->mValues[i]); VisitChild(initExpr->mValues[i]);
@ -1368,7 +1368,7 @@ void BfPrinter::Visit(BfGenericInstanceTypeRef* genericInstTypeRef)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(genericInstTypeRef->mCommas[i - 1]); VisitChildNoRef(genericInstTypeRef->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(genericInstTypeRef->mGenericArguments[i]); VisitChild(genericInstTypeRef->mGenericArguments[i]);
@ -1391,7 +1391,7 @@ void BfPrinter::Visit(BfTupleTypeRef* typeRef)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(typeRef->mCommas[i - 1]); VisitChildNoRef(typeRef->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(typeRef->mFieldTypes[i]); VisitChild(typeRef->mFieldTypes[i]);
@ -1421,7 +1421,7 @@ void BfPrinter::Visit(BfDelegateTypeRef* typeRef)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(typeRef->mCommas[i - 1]); VisitChildNoRef(typeRef->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(typeRef->mParams[i]); VisitChild(typeRef->mParams[i]);
@ -1573,7 +1573,7 @@ void BfPrinter::Visit(BfLambdaBindExpression* lambdaBindExpr)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(lambdaBindExpr->mCommas[i - 1]); VisitChildNoRef(lambdaBindExpr->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(lambdaBindExpr->mParams[i]); VisitChild(lambdaBindExpr->mParams[i]);
@ -1626,7 +1626,7 @@ void BfPrinter::Visit(BfObjectCreateExpression* newExpr)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(newExpr->mCommas[i - 1]); VisitChildNoRef(newExpr->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(newExpr->mArguments[i]); VisitChild(newExpr->mArguments[i]);
@ -1678,7 +1678,7 @@ void BfPrinter::Visit(BfInvocationExpression* invocationExpr)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(invocationExpr->mCommas[i - 1]); VisitChildNoRef(invocationExpr->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(invocationExpr->mArguments[i]); VisitChild(invocationExpr->mArguments[i]);
@ -1703,7 +1703,7 @@ void BfPrinter::Visit(BfDeferStatement* deferStmt)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(bind->mCommas[i - 1]); VisitChildNoRef(bind->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(bind->mParams[i]); VisitChild(bind->mParams[i]);
@ -1763,7 +1763,7 @@ void BfPrinter::Visit(BfSwitchCase* switchCase)
if ((caseIdx == 0) || (caseIdx > (int)switchCase->mCaseCommas.size())) if ((caseIdx == 0) || (caseIdx > (int)switchCase->mCaseCommas.size()))
ExpectSpace(); ExpectSpace();
else else
VisitChild(switchCase->mCaseCommas[caseIdx - 1]); VisitChildNoRef(switchCase->mCaseCommas.GetSafe(caseIdx - 1));
VisitChild(switchCase->mCaseExpressions[caseIdx]); VisitChild(switchCase->mCaseExpressions[caseIdx]);
} }
VisitChild(switchCase->mColonToken); VisitChild(switchCase->mColonToken);
@ -1987,7 +1987,7 @@ void BfPrinter::Visit(BfForStatement* forStmt)
for (int i = 0; i < (int) forStmt->mInitializers.size(); i++) for (int i = 0; i < (int) forStmt->mInitializers.size(); i++)
{ {
if (i > 0) if (i > 0)
VisitChild(forStmt->mInitializerCommas[i - 1]); VisitChildNoRef(forStmt->mInitializerCommas.GetSafe(i - 1));
VisitChild(forStmt->mInitializers[i]); VisitChild(forStmt->mInitializers[i]);
} }
VisitChild(forStmt->mInitializerSemicolon); VisitChild(forStmt->mInitializerSemicolon);
@ -1998,7 +1998,7 @@ void BfPrinter::Visit(BfForStatement* forStmt)
for (int i = 0; i < (int) forStmt->mIterators.size(); i++) for (int i = 0; i < (int) forStmt->mIterators.size(); i++)
{ {
if (i > 0) if (i > 0)
VisitChild(forStmt->mIteratorCommas[i - 1]); VisitChildNoRef(forStmt->mIteratorCommas.GetSafe(i - 1));
VisitChild(forStmt->mIterators[i]); VisitChild(forStmt->mIterators[i]);
} }
VisitChild(forStmt->mCloseParen); VisitChild(forStmt->mCloseParen);
@ -2086,7 +2086,7 @@ void BfPrinter::Visit(BfTupleExpression* tupleExpr)
if (i > 0) if (i > 0)
{ {
VisitChild(tupleExpr->mCommas[i - 1]); VisitChildNoRef(tupleExpr->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(tupleExpr->mValues[i]); VisitChild(tupleExpr->mValues[i]);
@ -2114,7 +2114,7 @@ void BfPrinter::Visit(BfIndexerExpression* indexerExpr)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(indexerExpr->mCommas[i - 1]); VisitChildNoRef(indexerExpr->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(indexerExpr->mArguments[i]); VisitChild(indexerExpr->mArguments[i]);
@ -2178,7 +2178,7 @@ void BfPrinter::Visit(BfConstructorDeclaration* ctorDeclaration)
{ {
if (i > 0) if (i > 0)
{ {
QueueVisitChild(ctorDeclaration->mCommas[i - 1]); QueueVisitChild(ctorDeclaration->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
QueueVisitChild(ctorDeclaration->mParams[i]); QueueVisitChild(ctorDeclaration->mParams[i]);
@ -2225,7 +2225,7 @@ void BfPrinter::Visit(BfDestructorDeclaration* dtorDeclaration)
{ {
if (i > 0) if (i > 0)
{ {
QueueVisitChild(dtorDeclaration->mCommas[i - 1]); QueueVisitChild(dtorDeclaration->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
QueueVisitChild(dtorDeclaration->mParams[i]); QueueVisitChild(dtorDeclaration->mParams[i]);
@ -2324,7 +2324,7 @@ void BfPrinter::QueueMethodDeclaration(BfMethodDeclaration* methodDeclaration)
{ {
if (i > 0) if (i > 0)
{ {
QueueVisitChild(methodDeclaration->mCommas[i - 1]); QueueVisitChild(methodDeclaration->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
QueueVisitChild(methodDeclaration->mParams[i]); QueueVisitChild(methodDeclaration->mParams[i]);
@ -2414,7 +2414,7 @@ void BfPrinter::Visit(BfPropertyDeclaration* propertyDeclaration)
{ {
if (i > 0) if (i > 0)
{ {
QueueVisitChild(indexerDeclaration->mCommas[i - 1]); QueueVisitChild(indexerDeclaration->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
QueueVisitChild(indexerDeclaration->mParams[i]); QueueVisitChild(indexerDeclaration->mParams[i]);
@ -2552,7 +2552,7 @@ void BfPrinter::Visit(BfEnumCaseDeclaration* enumCaseDeclaration)
{ {
if (i > 0) if (i > 0)
{ {
VisitChild(enumCaseDeclaration->mCommas[i - 1]); VisitChildNoRef(enumCaseDeclaration->mCommas.GetSafe(i - 1));
ExpectSpace(); ExpectSpace();
} }
VisitChild(enumCaseDeclaration->mEntries[i]); VisitChild(enumCaseDeclaration->mEntries[i]);
@ -2643,7 +2643,7 @@ void BfPrinter::Visit(BfTypeDeclaration* typeDeclaration)
QueueVisitChild(typeDeclaration->mBaseClasses[i]); QueueVisitChild(typeDeclaration->mBaseClasses[i]);
if (i > 0) if (i > 0)
{ {
QueueVisitChild(typeDeclaration->mBaseClassCommas[i - 1]); QueueVisitChild(typeDeclaration->mBaseClassCommas.GetSafe(i - 1));
} }
} }
ExpectSpace(); ExpectSpace();