From 9f2b2894ba2b45135943e943d13898aefc333cc2 Mon Sep 17 00:00:00 2001 From: Jason Zaugg Date: Sun, 16 Jun 2013 14:47:26 -0400 Subject: SI-7584 Test and spurious warning fix for by-name closures The enclosed test case exercises by-name closures, which were the subject of the previous commit. In the process, a spurious warning was eliminated. --- src/reflect/scala/reflect/internal/TreeInfo.scala | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/reflect') diff --git a/src/reflect/scala/reflect/internal/TreeInfo.scala b/src/reflect/scala/reflect/internal/TreeInfo.scala index a5cf46071f..cd1f5f2e47 100644 --- a/src/reflect/scala/reflect/internal/TreeInfo.scala +++ b/src/reflect/scala/reflect/internal/TreeInfo.scala @@ -208,7 +208,7 @@ abstract class TreeInfo { } def isWarnableSymbol = { val sym = tree.symbol - (sym == null) || !(sym.isModule || sym.isLazy) || { + (sym == null) || !(sym.isModule || sym.isLazy || definitions.isByNameParamType(sym.tpe_*)) || { debuglog("'Pure' but side-effecting expression in statement position: " + tree) false } -- cgit v1.2.3