Updated comments and formatting

Author: elsh

include/swift/AST/Decl.h

@@ -1132,10 +1132,9 @@ class ImportDecl final : public Decl,
 
   SourceLoc ImportLoc;
   SourceLoc KindLoc;
-  // Used to store the real module name corresponding to this import decl
-  // in case module aliasing is used. For example if '-module-alias Foo=Bar'
-  // was passed and this decl is 'import Foo', the real name 'Bar' will be
-  // stored.
+  /// Used to store the real module name corresponding to this import decl in
+  /// case module aliasing is used. For example if '-module-alias Foo=Bar' was
+  /// passed and this decl is 'import Foo', the real name 'Bar' will be stored.
   Identifier RealModuleName;
 
   /// The resolved module.
@@ -1168,39 +1167,36 @@ class ImportDecl final : public Decl,
 
   /// Retrieves the import path as written in the source code.
   /// 
-  /// \returns An \c ImportPath corresponding to this import decl. If module
-  ///          aliasing was used, this will contain the aliased name of the module;
-  ///          for instance, if you wrote 'import Foo' but passed
-  ///          '-module-alias Foo=Bar', this import path will include 'Foo'. This
-  ///          return value is always owned by the AST context, so it can be
-  ///          persisted.
+  /// \returns An \c ImportPath corresponding to this import decl. If module aliasing
+  ///          was used, this will contain the aliased name of the module; for instance,
+  ///          if you wrote 'import Foo' but passed '-module-alias Foo=Bar', this import
+  ///          path will include 'Foo'. This return value is always owned by \c ImportDecl
+  ///          (which is owned by the AST context), so it can be persisted.
   ImportPath getImportPath() const {
-     return ImportPath({ getTrailingObjects<ImportPath::Element>(),
-                         static_cast<size_t>(Bits.ImportDecl.NumPathElements) });
+    return ImportPath({ getTrailingObjects<ImportPath::Element>(),
+                        static_cast<size_t>(Bits.ImportDecl.NumPathElements) });
   }
 
   /// Retrieves the import path, replacing any module aliases with real names.
   /// 
   /// \param scratch An \c ImportPath::Builder which may, if necessary, be used to
   ///        construct the return value. It may go unused, so you should not try to
   ///        read the result from it; use the return value instead.
-  /// \returns An \c ImportPath corresponding to this import decl. If module
-  ///          aliasing was used, this will contain the real name of the module;
-  ///          for instance, if you wrote 'import Foo' but passed
-  ///          '-module-alias Foo=Bar', this import path will include 'Bar'. This
-  ///          return value may be owned by \p scratch, so it should not be used
-  ///          after \p scratch is destroyed.
+  /// \returns An \c ImportPath corresponding to this import decl. If module aliasing
+  ///          was used, this will contain the real name of the module; for instance,
+  ///          if you wrote 'import Foo' but passed '-module-alias Foo=Bar', this import
+  ///          path will include 'Bar'. This return value may be owned by \p scratch,
+  ///          so it should not be used after \p scratch is destroyed.
   ImportPath getRealImportPath(ImportPath::Builder &scratch) const;
 
   /// Retrieves the part of the import path that contains the module name,
   /// as written in the source code.
   /// 
-  /// \returns A \c ModulePath corresponding to this import decl. If module
-  ///          aliasing was used, this will contain the aliased name of the module;
-  ///          for instance, if you wrote 'import Foo' but passed
-  ///          '-module-alias Foo=Bar', this module path will include 'Foo'. This
-  ///          return value is always owned by the AST context, so it can be
-  ///          persisted.
+  /// \returns A \c ImportPath::Module corresponding to this import decl. If module
+  ///          aliasing was used, this will contain the aliased name of the module; for
+  ///          instance, if you wrote 'import Foo' but passed '-module-alias Foo=Bar',
+  ///          this module path will contain 'Foo'. This return value is always owned by
+  ///          \c ImportDecl (which is owned by the AST context), so it can be persisted.
   ImportPath::Module getModulePath() const {
     return getImportPath().getModulePath(getImportKind());
   }
@@ -1211,12 +1207,11 @@ class ImportDecl final : public Decl,
   /// \param scratch An \c ImportPath::Builder which may, if necessary, be used to
   ///        construct the return value. It may go unused, so you should not try to
   ///        read the result from it; use the return value instead.
-  /// \returns An \c ImportPath corresponding to this import decl. If module
-  ///          aliasing was used, this will contain the real name of the module;
-  ///          for instance, if you wrote 'import Foo' but passed
-  ///          '-module-alias Foo=Bar', this import path will include 'Bar'. This
-  ///          return value may be owned by \p scratch, so it should not be used
-  ///          after \p scratch is destroyed.
+  /// \returns An \c ImportPath::Module corresponding to this import decl. If module
+  ///          aliasing was used, this will contain the real name of the module; for
+  ///          instance, if you wrote 'import Foo' but passed '-module-alias Foo=Bar',
+  ///          the returned path will contain 'Bar'. This return value may be owned
+  ///          by \p scratch, so it should not be used after \p scratch is destroyed.
   ImportPath::Module getRealModulePath(ImportPath::Builder &scratch) const {
     return getRealImportPath(scratch).getModulePath(getImportKind());
   }

lib/AST/Decl.cpp

@@ -1066,7 +1066,8 @@ ImportDecl *ImportDecl::create(ASTContext &Ctx, DeclContext *DC,
   auto D = new (ptr) ImportDecl(DC, ImportLoc, Kind, KindLoc, Path);
   if (ClangN)
     D->setClangNode(ClangN);
-  auto realNameIfExists = Ctx.getRealModuleName(Path.front().Item, ASTContext::ModuleAliasLookupOption::realNameFromAlias);
+  auto realNameIfExists = Ctx.getRealModuleName(Path.front().Item,
+                                                ASTContext::ModuleAliasLookupOption::realNameFromAlias);
   if (!realNameIfExists.empty()) {
     D->RealModuleName = realNameIfExists;
   }