[Bast-commits] r4950 - in Devel-Declare/1.000/trunk: . lib/Devel

rafl at dev.catalyst.perl.org rafl at dev.catalyst.perl.org
Wed Oct 22 19:29:36 BST 2008


Author: rafl
Date: 2008-10-22 19:29:36 +0100 (Wed, 22 Oct 2008)
New Revision: 4950

Modified:
   Devel-Declare/1.000/trunk/Declare.xs
   Devel-Declare/1.000/trunk/Makefile.PL
   Devel-Declare/1.000/trunk/lib/Devel/Declare.pm
Log:
Merge branch 'hook_op_check'

* hook_op_check:
  Update for latest B::Hooks::OP::Check API.
  Use B::Hooks::OP::Check to register PL_check callbacks.
  Create branch hook_op_check

Modified: Devel-Declare/1.000/trunk/Declare.xs
===================================================================
--- Devel-Declare/1.000/trunk/Declare.xs	2008-10-22 18:27:57 UTC (rev 4949)
+++ Devel-Declare/1.000/trunk/Declare.xs	2008-10-22 18:29:36 UTC (rev 4950)
@@ -1,6 +1,7 @@
 #include "EXTERN.h"
 #include "perl.h"
 #include "XSUB.h"
+#include "hook_op_check.h"
 #undef printf
 #include "stolen_chunk_of_toke.c"
 #include <stdio.h>
@@ -192,14 +193,10 @@
 
 /* replacement PL_check rv2cv entry */
 
-STATIC OP *(*dd_old_ck_rv2cv)(pTHX_ OP *op);
-
-STATIC OP *dd_ck_rv2cv(pTHX_ OP *o) {
+STATIC OP *dd_ck_rv2cv(pTHX_ OP *o, void *user_data) {
   OP* kid;
   int dd_flags;
 
-  o = dd_old_ck_rv2cv(aTHX_ o); /* let the original do its job */
-
   if (in_declare) {
     if (dd_debug) {
       printf("Deconstructing declare\n");
@@ -258,8 +255,6 @@
   return o;
 }
 
-STATIC OP *(*dd_old_ck_entereval)(pTHX_ OP *op);
-
 OP* dd_pp_entereval(pTHX) {
   dSP;
   dPOPss;
@@ -284,8 +279,7 @@
   return PL_ppaddr[OP_ENTEREVAL](aTHX);
 }
 
-STATIC OP *dd_ck_entereval(pTHX_ OP *o) {
-  o = dd_old_ck_entereval(aTHX_ o); /* let the original do its job */
+STATIC OP *dd_ck_entereval(pTHX_ OP *o, void *user_data) {
   if (o->op_ppaddr == PL_ppaddr[OP_ENTEREVAL])
     o->op_ppaddr = dd_pp_entereval;
   return o;
@@ -299,14 +293,10 @@
   return count;
 }
 
-STATIC OP *(*dd_old_ck_const)(pTHX_ OP*op);
-
-STATIC OP *dd_ck_const(pTHX_ OP *o) {
+STATIC OP *dd_ck_const(pTHX_ OP *o, void *user_data) {
   int dd_flags;
   char* name;
 
-  o = dd_old_ck_const(aTHX_ o); /* let the original do its job */
-
   /* if this is set, we just grabbed a delimited string or something,
      not a bareword, so NO TOUCHY */
 
@@ -339,12 +329,9 @@
 setup()
   CODE:
   if (!initialized++) {
-    dd_old_ck_rv2cv = PL_check[OP_RV2CV];
-    PL_check[OP_RV2CV] = dd_ck_rv2cv;
-    dd_old_ck_entereval = PL_check[OP_ENTEREVAL];
-    PL_check[OP_ENTEREVAL] = dd_ck_entereval;
-    dd_old_ck_const = PL_check[OP_CONST];
-    PL_check[OP_CONST] = dd_ck_const;
+    hook_op_check(OP_RV2CV, dd_ck_rv2cv, NULL);
+    hook_op_check(OP_ENTEREVAL, dd_ck_entereval, NULL);
+    hook_op_check(OP_CONST, dd_ck_const, NULL);
   }
   filter_add(dd_filter_realloc, NULL);
 

Modified: Devel-Declare/1.000/trunk/Makefile.PL
===================================================================
--- Devel-Declare/1.000/trunk/Makefile.PL	2008-10-22 18:27:57 UTC (rev 4949)
+++ Devel-Declare/1.000/trunk/Makefile.PL	2008-10-22 18:29:36 UTC (rev 4950)
@@ -2,19 +2,28 @@
 
 use 5.008001;
 
+use ExtUtils::Depends;
+
 name 'Devel-Declare';
 all_from 'lib/Devel/Declare.pm';
 
 requires 'Scalar::Util';
+requires 'B::Hooks::OP::Check' => '0.10';
 build_requires 'Test::More';
 build_requires 'Scope::Guard';
+build_requires 'B::Hooks::OP::Check' => '0.10';
+configure_requires 'ExtUtils::Depends';
+configure_requires 'B::Hooks::OP::Check' => '0.10';
 
 postamble(<<'EOM');
 $(OBJECT) : stolen_chunk_of_toke.c
 EOM
 
+my $pkg = ExtUtils::Depends->new('Devel::Declare', 'B::Hooks::OP::Check');
+
 WriteMakefile(
   dist => {
     PREOP => 'pod2text lib/Devel/Declare.pm >$(DISTVNAME)/README'
-  }
+  },
+  $pkg->get_makefile_vars,
 );

Modified: Devel-Declare/1.000/trunk/lib/Devel/Declare.pm
===================================================================
--- Devel-Declare/1.000/trunk/lib/Devel/Declare.pm	2008-10-22 18:27:57 UTC (rev 4949)
+++ Devel-Declare/1.000/trunk/lib/Devel/Declare.pm	2008-10-22 18:29:36 UTC (rev 4950)
@@ -14,6 +14,7 @@
 use vars qw(%declarators %declarator_handlers @ISA);
 use base qw(DynaLoader);
 use Scalar::Util 'set_prototype';
+use B::Hooks::OP::Check;
 
 bootstrap Devel::Declare;
 




More information about the Bast-commits mailing list