Ensure spaces after if in Dotty source.

Author: DarkDimius

project/Build.scala

@@ -109,7 +109,7 @@ object DottyBuild extends Build {
       val path = for {
         file <- attList.map(_.data)
         path = file.getAbsolutePath
-        prefix = if(path.endsWith(".jar")) "p" else "a"
+        prefix = if (path.endsWith(".jar")) "p" else "a"
       } yield "-Xbootclasspath/" + prefix + ":" + path
       // dotty itself needs to be in the bootclasspath
       val fullpath = ("-Xbootclasspath/a:" + bin) :: path.toList

src/dotty/tools/backend/jvm/CollectEntryPoints.scala

@@ -40,7 +40,7 @@ class CollectEntryPoints extends MiniPhaseTransform {
   def phaseName: String = "Collect entry points"
 
   override def transformDefDef(tree: tpd.DefDef)(implicit ctx: Context, info: TransformerInfo): tpd.Tree = {
-    if((tree.symbol ne NoSymbol) && CollectEntryPoints.isJavaEntyPoint(tree.symbol)) {
+    if ((tree.symbol ne NoSymbol) && CollectEntryPoints.isJavaEntyPoint(tree.symbol)) {
       ctx.genBCodePhase.asInstanceOf[GenBCode].registerEntryPoint(tree.symbol)
     }
     tree

src/dotty/tools/backend/jvm/DottyBackendInterface.scala

@@ -376,7 +376,7 @@ class DottyBackendInterface()(implicit ctx: Context) extends BackendInterface{
   def shouldEmitJumpAfterLabels = true
 
   def dumpClasses: Option[String] =
-    if(ctx.settings.Ydumpclasses.isDefault) None
+    if (ctx.settings.Ydumpclasses.isDefault) None
     else Some(ctx.settings.Ydumpclasses.value)
 
   def mainClass: Option[String] =
@@ -423,7 +423,7 @@ class DottyBackendInterface()(implicit ctx: Context) extends BackendInterface{
       case TermRef(prefix: ThisType, name) =>
         Some(tpd.This(prefix.cls).select(i.symbol))
       case TermRef(NoPrefix, name) =>
-        if(i.symbol is Flags.Method) Some(This(i.symbol.enclosingClass).select(i.symbol)) // workaround #342 todo: remove after fixed
+        if (i.symbol is Flags.Method) Some(This(i.symbol.enclosingClass).select(i.symbol)) // workaround #342 todo: remove after fixed
         else None
       case _ => None
     }
@@ -663,7 +663,7 @@ class DottyBackendInterface()(implicit ctx: Context) extends BackendInterface{
     def companionSymbol: Symbol = if (sym is Flags.Module) companionClass else companionModule
     def moduleClass: Symbol = toDenot(sym).moduleClass
     def enclosingClassSym: Symbol = {
-      if(this.isClass) {
+      if (this.isClass) {
         val ct = ctx.withPhase(ctx.flattenPhase.prev)
         toDenot(sym)(ct).owner.enclosingClass(ct)
       }
@@ -792,7 +792,7 @@ class DottyBackendInterface()(implicit ctx: Context) extends BackendInterface{
           t.info match {
 
             case _ =>
-              if(!t.symbol.isClass) nonClassTypeRefToBType(t.symbol)  // See comment on nonClassTypeRefToBType
+              if (!t.symbol.isClass) nonClassTypeRefToBType(t.symbol)  // See comment on nonClassTypeRefToBType
               else primitiveOrClassToBType(t.symbol) // Common reference to a type such as scala.Int or java.lang.String
           }
         case Types.ClassInfo(_, sym, _, _, _)           => primitiveOrClassToBType(sym) // We get here, for example, for genLoadModule, which invokes toTypeKind(moduleClassSymbol.info)
@@ -942,7 +942,7 @@ class DottyBackendInterface()(implicit ctx: Context) extends BackendInterface{
     def _3: Tree = field.rhs
 
     override def unapply(s: LabelDef): DottyBackendInterface.this.LabelDef.type = {
-      if(s.symbol is Flags.Label) this.field = s
+      if (s.symbol is Flags.Label) this.field = s
       else this.field = null
       this
     }
@@ -1021,11 +1021,11 @@ class DottyBackendInterface()(implicit ctx: Context) extends BackendInterface{
     def _2 = field.meth
     def _3 = {
       val t = field.tpt.tpe.typeSymbol
-      if(t.exists) t
+      if (t.exists) t
       else {
         val arity = field.meth.tpe.widenDealias.paramTypes.size - _1.size
         val returnsUnit = field.meth.tpe.widenDealias.resultType.classSymbol == UnitClass
-        if(returnsUnit)
+        if (returnsUnit)
           ctx.requiredClass(("scala.compat.java8.JProcedure" + arity).toTermName)
         else ctx.requiredClass(("scala.compat.java8.JFunction" + arity).toTermName)
       }

src/dotty/tools/backend/jvm/LabelDefs.scala

@@ -44,7 +44,7 @@ import StdNames.nme
  *
  * <label> def foo(i: Int) = {
  *   <label> def bar = 0
- *   <label> def dough(i: Int) = if(i == 0) bar else foo(i-1)
+ *   <label> def dough(i: Int) = if (i == 0) bar else foo(i-1)
  *   dough(i)
  *   }
  *
@@ -54,7 +54,7 @@ import StdNames.nme
  *
  *                                                  \
  * <label> def foo(i: Int) = dough(i)
- * <label> def dough(i: Int) = if(i == 0) bar else foo(i-1)
+ * <label> def dough(i: Int) = if (i == 0) bar else foo(i-1)
  * <label> def bar = 2
  *   foo(100)
  *
@@ -64,7 +64,7 @@ import StdNames.nme
  *  <jump foo>
  *  <label> def foo(i: Int) = dough(i)
  *  // <jump a>                           // unreachable
- *  <label> def dough(i: Int) = if(i == 0) bar else foo(i-1)
+ *  <label> def dough(i: Int) = if (i == 0) bar else foo(i-1)
  *  // <jump a>                           // unreachable
  *  <label> def bar = 2
  *  // <jump a>                           // unreachable
@@ -107,7 +107,7 @@ class LabelDefs extends MiniPhaseTransform {
               labelLevel = labelLevel + 1
               val r = Block(moveLabels(t), t)
               labelLevel = labelLevel - 1
-              if(labelLevel == 0) beingAppended.clear()
+              if (labelLevel == 0) beingAppended.clear()
               r
             case _ => if (entryPoints.nonEmpty && labelDefs.nonEmpty) super.transform(tree) else tree
           }
@@ -206,14 +206,14 @@ class LabelDefs extends MiniPhaseTransform {
         labelCalls(r.symbol) = parentLabelCalls
         parentLabelCalls = st
 
-        if(shouldMoveLabel) {
+        if (shouldMoveLabel) {
           labelDefs(r.symbol) = r
           EmptyTree
         } else r
       case t: Apply if t.symbol is Flags.Label =>
         val sym = t.symbol
         parentLabelCalls = parentLabelCalls + t
-        if(owner != sym) callCounts(sym) = callCounts(sym) + 1
+        if (owner != sym) callCounts(sym) = callCounts(sym) + 1
         super.transform(tree)
       case _ =>
         super.transform(tree)

src/dotty/tools/dotc/ast/tpd.scala

@@ -776,7 +776,7 @@ object tpd extends Trees.Instance[Type] with TypedTreeInfo {
       } else args
     }
 
-    val callArgs: List[Tree] = if(args.isEmpty) Nil else {
+    val callArgs: List[Tree] = if (args.isEmpty) Nil else {
       val expectedType = selected.widen.paramTypess.head.last
       val lastParam = args.last
       adaptLastArg(lastParam, expectedType)

src/dotty/tools/dotc/core/Symbols.scala

@@ -162,7 +162,7 @@ trait Symbols { this: Context =>
         privateWithin, coord, assocFile)
 
   def synthesizeCompanionMethod(name: Name, target: SymDenotation, owner: SymDenotation)(implicit ctx: Context) =
-    if(owner.exists && target.exists && !owner.isAbsent && !target.isAbsent) {
+    if (owner.exists && target.exists && !owner.isAbsent && !target.isAbsent) {
       val existing = owner.unforcedDecls.lookup(name)
 
       existing.orElse{

src/dotty/tools/dotc/core/TypeErasure.scala

@@ -95,7 +95,7 @@ object TypeErasure {
   def erasure(tp: Type)(implicit ctx: Context): Type = scalaErasureFn(tp)(erasureCtx)
   def semiErasure(tp: Type)(implicit ctx: Context): Type = semiErasureFn(tp)(erasureCtx)
   def sigName(tp: Type, isJava: Boolean)(implicit ctx: Context): TypeName = {
-    val seqClass = if(isJava) defn.ArrayClass else defn.SeqClass
+    val seqClass = if (isJava) defn.ArrayClass else defn.SeqClass
     val normTp =
       if (tp.isRepeatedParam) tp.translateParameterized(defn.RepeatedParamClass, seqClass)
       else tp

src/dotty/tools/dotc/core/Types.scala

@@ -76,7 +76,7 @@ object Types {
 
     val uniqId = {
       nextId = nextId + 1
-//      if(nextId == 19555)
+//      if (nextId == 19555)
 //        println("foo")
       nextId
     }

src/dotty/tools/dotc/core/pickling/ClassfileParser.scala

@@ -92,7 +92,7 @@ class ClassfileParser(
       if (c != classRoot.symbol) mismatchError(c)
     }
 
-    if(classRoot.symbol.id == 4812) {
+    if (classRoot.symbol.id == 4812) {
       println("bar")
     }
 

src/dotty/tools/dotc/parsing/JavaParsers.scala

@@ -121,11 +121,11 @@ object JavaParsers {
         case nil => (EmptyTree, nil)
       }
       var (constr1, stats1) = pullOutFirstConstr(stats)
-      if(constr1 == EmptyTree) constr1 = makeConstructor(List(), tparams)
+      if (constr1 == EmptyTree) constr1 = makeConstructor(List(), tparams)
       // A dummy first constructor is needed for Java classes so that the real constructors see the
       // import of the companion object. The constructor has parameter of type Unit so no Java code
       // can call it.
-      if(needsDummyConstr) {
+      if (needsDummyConstr) {
         stats1 = constr1 :: stats1
         constr1 = makeConstructor(List(scalaDot(tpnme.Unit)), tparams, Flags.JavaDefined | Flags.PrivateLocal)
       }
@@ -579,9 +579,9 @@ object JavaParsers {
     def varDecl(pos: Position, mods: Modifiers, tpt: Tree, name: TermName): ValDef = {
       val tpt1 = optArrayBrackets(tpt)
       if (in.token == EQUALS && !(mods is Flags.Param)) skipTo(COMMA, SEMI)
-      val mods1 = if(mods is Flags.Final) mods else mods | Flags.Mutable
+      val mods1 = if (mods is Flags.Final) mods else mods | Flags.Mutable
       atPos(pos) {
-        ValDef(name, tpt1, if(mods is Flags.Param) EmptyTree else unimplementedExpr).withMods(mods1)
+        ValDef(name, tpt1, if (mods is Flags.Param) EmptyTree else unimplementedExpr).withMods(mods1)
       }
     }