aboutsummaryrefslogtreecommitdiff
path: root/lib/Sema/SemaInit.cpp
diff options
context:
space:
mode:
authorMike Stump <mrs@apple.com>2009-05-16 07:39:55 +0000
committerMike Stump <mrs@apple.com>2009-05-16 07:39:55 +0000
commit390b4cc8b45a05612349269ef08faab3e4688f06 (patch)
treeb98456ec626f3ed6d88aa89d02e279617514faa0 /lib/Sema/SemaInit.cpp
parentc6e35aae23bc3cea7daf5ee075fa695c01c0f66f (diff)
Reflow some comments.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@71936 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Sema/SemaInit.cpp')
-rw-r--r--lib/Sema/SemaInit.cpp34
1 files changed, 16 insertions, 18 deletions
diff --git a/lib/Sema/SemaInit.cpp b/lib/Sema/SemaInit.cpp
index 36b0e3b135..a8a0ea3aea 100644
--- a/lib/Sema/SemaInit.cpp
+++ b/lib/Sema/SemaInit.cpp
@@ -174,8 +174,8 @@ bool Sema::CheckInitializerTypes(Expr *&Init, QualType &DeclType,
// expression as its argument; if the function is a
// constructor, the call initializes a temporary of the
// destination type.
- // FIXME: We're pretending to do copy elision here; return to
- // this when we have ASTs for such things.
+ // FIXME: We're pretending to do copy elision here; return to this when we
+ // have ASTs for such things.
if (!PerformImplicitConversion(Init, DeclType, "initializing"))
return false;
@@ -349,10 +349,9 @@ void InitListChecker::FillInValueInitializations(InitListExpr *ILE) {
return;
}
- // FIXME: If value-initialization involves calling a
- // constructor, should we make that call explicit in the
- // representation (even when it means extending the
- // initializer list)?
+ // FIXME: If value-initialization involves calling a constructor, should
+ // we make that call explicit in the representation (even when it means
+ // extending the initializer list)?
if (Init < NumInits && !hadError)
ILE->setInit(Init,
new (SemaRef.Context) ImplicitValueInitExpr(Field->getType()));
@@ -390,10 +389,9 @@ void InitListChecker::FillInValueInitializations(InitListExpr *ILE) {
return;
}
- // FIXME: If value-initialization involves calling a
- // constructor, should we make that call explicit in the
- // representation (even when it means extending the
- // initializer list)?
+ // FIXME: If value-initialization involves calling a constructor, should
+ // we make that call explicit in the representation (even when it means
+ // extending the initializer list)?
if (Init < NumInits && !hadError)
ILE->setInit(Init,
new (SemaRef.Context) ImplicitValueInitExpr(ElementType));
@@ -761,10 +759,10 @@ void InitListChecker::CheckReferenceType(InitListExpr *IList, QualType DeclType,
UpdateStructuredListElement(StructuredList, StructuredIndex, expr);
++Index;
} else {
- // FIXME: It would be wonderful if we could point at the actual
- // member. In general, it would be useful to pass location
- // information down the stack, so that we know the location (or
- // decl) of the "current object" being initialized.
+ // FIXME: It would be wonderful if we could point at the actual member. In
+ // general, it would be useful to pass location information down the stack,
+ // so that we know the location (or decl) of the "current object" being
+ // initialized.
SemaRef.Diag(IList->getLocStart(),
diag::err_init_reference_member_uninitialized)
<< DeclType
@@ -1733,8 +1731,8 @@ bool Sema::CheckValueInitialization(QualType Type, SourceLocation Loc) {
// called (and the initialization is ill-formed if T has no
// accessible default constructor);
if (ClassDecl->hasUserDeclaredConstructor())
- // FIXME: Eventually, we'll need to put the constructor decl
- // into the AST.
+ // FIXME: Eventually, we'll need to put the constructor decl into the
+ // AST.
return PerformInitializationByConstructor(Type, 0, 0, Loc,
SourceRange(Loc),
DeclarationName(),
@@ -1747,8 +1745,8 @@ bool Sema::CheckValueInitialization(QualType Type, SourceLocation Loc) {
// [...] A program that calls for default-initialization or
// value-initialization of an entity of reference type is
// ill-formed. [...]
- // FIXME: Once we have code that goes through this path, add an
- // actual diagnostic :)
+ // FIXME: Once we have code that goes through this path, add an actual
+ // diagnostic :)
}
return false;