Collapse `LocalDef` into `T{...}` macro, make it more flexible so it only caches `def`s (and complains about `val`s) if it's defined top-level in a class/trait/object, for easy overriding.

If it's in some helper function somewhere, overriding doesn't matter, and it's up to the user to figure out how to construct a target graph without redundancy

1 files changed, 1 insertions, 4 deletions

diff --git a/core/src/test/scala/forge/CacherTests.scala b/core/src/test/scala/forge/CacherTests.scala
index c6157a4a..de8265bd 100644
--- a/core/src/test/scala/forge/CacherTests.scala
+++ b/core/src/test/scala/forge/CacherTests.scala
@@ -48,13 +48,10 @@ object CacherTests extends TestSuite{
     )
     'errors{
       val expectedMsg =
-        "T{} can only be used directly within a zero-arg method defined in a class body"
+        "T{} members defined in a Cacher class/trait/object body must be defs"
 
       val err1 = compileError("object Foo extends Target.Cacher{ val x = T{1} }")
       assert(err1.msg == expectedMsg)
-
-      val err2 = compileError("object Foo extends Target.Cacher{ def x = {def y = T{1}} }")
-      assert(err2.msg == expectedMsg)
     }
   }
 }