Merge pull request #584 from hughsimpson/tidier_annotation_check

tidy up hasCacheKeyExcludeAnnotation
cb372 · Nov 9, 2021 · 9411622 · 9411622
2 parents 55e8cdc + fac28f1
commit 9411622
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/modules/core/src/main/scala-3/scalacache/memoization/Macros.scala b/modules/core/src/main/scala-3/scalacache/memoization/Macros.scala
@@ -32,9 +32,8 @@ object Macros {
     val sym: Symbol = Symbol.spliceOwner
 
     def hasCacheKeyExcludeAnnotation(s: Symbol): Boolean = s.annotations.exists {
-//      case Apply(Select(New(Ident("cacheKeyExclude")), _), _) => true // Don't know why this doesn't match...
-      case Apply(Select(New(o), _), _) => o.toString == "Ident(cacheKeyExclude)"
-      case o                           => false
+      case Apply(Select(New(TypeIdent("cacheKeyExclude")), _), _) => true
+      case o                                                      => false
     }
 
     def getOwningDefSymbol(s: Symbol): Symbol =