diff options
author | Jason Zaugg <jzaugg@gmail.com> | 2013-02-25 00:25:51 +0100 |
---|---|---|
committer | Jason Zaugg <jzaugg@gmail.com> | 2013-02-25 01:48:55 +0100 |
commit | 256e46824636881f067ea0d312b5cbcdffbcf233 (patch) | |
tree | 46ff04f70122f615a4af22a7bd7c8200ba395f88 /src/compiler | |
parent | bc997702ef50d8621d12f855ee17819d89c542f1 (diff) | |
download | scala-256e46824636881f067ea0d312b5cbcdffbcf233.tar.gz scala-256e46824636881f067ea0d312b5cbcdffbcf233.tar.bz2 scala-256e46824636881f067ea0d312b5cbcdffbcf233.zip |
Remove redundant explicit returns.
Diffstat (limited to 'src/compiler')
5 files changed, 8 insertions, 8 deletions
diff --git a/src/compiler/scala/reflect/reify/codegen/GenTypes.scala b/src/compiler/scala/reflect/reify/codegen/GenTypes.scala index 2370f18e3a..d389f3571b 100644 --- a/src/compiler/scala/reflect/reify/codegen/GenTypes.scala +++ b/src/compiler/scala/reflect/reify/codegen/GenTypes.scala @@ -155,7 +155,7 @@ trait GenTypes { */ private def reifySemiConcreteTypeMember(tpe: Type): Tree = tpe match { case tpe @ TypeRef(pre @ SingleType(prepre, presym), sym, args) if sym.isAbstractType && !sym.isExistential => - return mirrorFactoryCall(nme.TypeRef, reify(pre), mirrorBuildCall(nme.selectType, reify(sym.owner), reify(sym.name.toString)), reify(args)) + mirrorFactoryCall(nme.TypeRef, reify(pre), mirrorBuildCall(nme.selectType, reify(sym.owner), reify(sym.name.toString)), reify(args)) } /** Reify an annotated type, i.e. the one that makes us deal with AnnotationInfos */ diff --git a/src/compiler/scala/tools/ant/Scaladoc.scala b/src/compiler/scala/tools/ant/Scaladoc.scala index 7fc811788e..5c21399092 100644 --- a/src/compiler/scala/tools/ant/Scaladoc.scala +++ b/src/compiler/scala/tools/ant/Scaladoc.scala @@ -78,7 +78,7 @@ class Scaladoc extends ScalaMatchingTask { val values = List("yes", "no", "on", "off") def getBooleanValue(value: String, flagName: String): Boolean = if (Flag.isPermissible(value)) - return ("yes".equals(value) || "on".equals(value)) + ("yes".equals(value) || "on".equals(value)) else buildError("Unknown " + flagName + " flag '" + value + "'") } diff --git a/src/compiler/scala/tools/nsc/backend/icode/Primitives.scala b/src/compiler/scala/tools/nsc/backend/icode/Primitives.scala index 5eceb1cf6b..4fa717309e 100644 --- a/src/compiler/scala/tools/nsc/backend/icode/Primitives.scala +++ b/src/compiler/scala/tools/nsc/backend/icode/Primitives.scala @@ -230,9 +230,9 @@ trait Primitives { self: ICodes => /** Returns a string representation of this operation. */ override def toString(): String = this match { - case AND => return "AND" - case OR => return "OR" - case XOR => return "XOR" + case AND => "AND" + case OR => "OR" + case XOR => "XOR" case _ => throw new RuntimeException("LogicalOp unknown case") } } diff --git a/src/compiler/scala/tools/nsc/interpreter/AbstractFileClassLoader.scala b/src/compiler/scala/tools/nsc/interpreter/AbstractFileClassLoader.scala index 9fbd337b9d..3b272aee32 100644 --- a/src/compiler/scala/tools/nsc/interpreter/AbstractFileClassLoader.scala +++ b/src/compiler/scala/tools/nsc/interpreter/AbstractFileClassLoader.scala @@ -52,7 +52,7 @@ class AbstractFileClassLoader(val root: AbstractFile, parent: ClassLoader) return null } - return file + file } // parent delegation in JCL uses getResource; so either add parent.getResAsStream diff --git a/src/compiler/scala/tools/nsc/transform/ExplicitOuter.scala b/src/compiler/scala/tools/nsc/transform/ExplicitOuter.scala index 965612f926..124dd6c995 100644 --- a/src/compiler/scala/tools/nsc/transform/ExplicitOuter.scala +++ b/src/compiler/scala/tools/nsc/transform/ExplicitOuter.scala @@ -481,14 +481,14 @@ abstract class ExplicitOuter extends InfoTransform // at least don't crash... this duplicates maybeOmittable from constructors (acc.owner.isEffectivelyFinal && !acc.isOverridingSymbol)) { unit.uncheckedWarning(tree.pos, "The outer reference in this type test cannot be checked at run time.") - return transform(TRUE) // urgh... drop condition if there's no accessor (or if it may disappear after constructors) + transform(TRUE) // urgh... drop condition if there's no accessor (or if it may disappear after constructors) } else { // println("(base, acc)= "+(base, acc)) val outerSelect = localTyper typed Apply(Select(base, acc), Nil) // achieves the same as: localTyper typed atPos(tree.pos)(outerPath(base, base.tpe.typeSymbol, outerFor.outerClass)) // println("(b, tpsym, outerForI, outerFor, outerClass)= "+ (base, base.tpe.typeSymbol, outerFor, sel.symbol.owner, outerFor.outerClass)) // println("outerSelect = "+ outerSelect) - return transform(treeCopy.Apply(tree, treeCopy.Select(eqsel, outerSelect, eq), args)) + transform(treeCopy.Apply(tree, treeCopy.Select(eqsel, outerSelect, eq), args)) } case _ => |