pjfanning commented on code in PR #1968: URL: https://github.com/apache/pekko/pull/1968#discussion_r2239199089
########## actor-typed/src/main/scala/org/apache/pekko/actor/typed/internal/LoggerClass.scala: ########## @@ -26,6 +27,7 @@ import pekko.util.OptionVal private[pekko] object LoggerClass { // just to get access to the class context + @nowarn("msg=deprecated") Review Comment: The whole SecurityManager class is deprecated (since Java 19). We might need to change this to get the context classes from a stack trace instead. If you see the StackBench change in this PR, it seems like there is a perf benefit (or maybe used to be a benefit) of doing it with the SecurityManager. I logged #1970 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: notifications-unsubscr...@pekko.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: notifications-unsubscr...@pekko.apache.org For additional commands, e-mail: notifications-h...@pekko.apache.org