Branch: refs/heads/blead Home: https://github.com/Perl/perl5 Commit: 981e5d62ca8690e324d97819c14e8b722e3ee3bb https://github.com/Perl/perl5/commit/981e5d62ca8690e324d97819c14e8b722e3ee3bb Author: Tony Cook <t...@develop-help.com> Date: 2023-09-25 (Mon, 25 Sep 2023)
Changed paths: M t/op/eval.t Log Message: ----------- add TODO tests for 19370 Commit: 386907f061c1812ecaa5f3c88d9f729828408097 https://github.com/Perl/perl5/commit/386907f061c1812ecaa5f3c88d9f729828408097 Author: Tony Cook <t...@develop-help.com> Date: 2023-09-25 (Mon, 25 Sep 2023) Changed paths: M cv.h M dump.c M ext/Devel-Peek/t/Peek.t M lib/B/Deparse.t M pad.c M t/op/closure.t M t/op/eval.t Log Message: ----------- Revert "[perl #89544] Non-eval closures don’t need CvOUTSIDE" But they do need CvOUTSIDE() for eval-in-package-DB's scope magic to work correctly. This also incidentally fixes a TODO Deparse, since the CvOUTSIDE is now available Fixes #19370 aka rt89544 This breaks #11286, but I don't think that's fixable without breaking eval-in-DB's special scope behaviour. This reverts (most of) commit a0d2bbd5c47035a4f7369e4fddd46b502764d86e. Compare: https://github.com/Perl/perl5/compare/fc5e06749eb0...386907f061c1