/* pad.h
*
- * Copyright (c) 2002, Larry Wall
+ * Copyright (C) 2002, by Larry Wall and others
*
* You may distribute under the terms of either the GNU General Public
* License or the Artistic License, as specified in the README file.
/* flags for the pad_new() function */
-typedef enum {
- padnew_CLONE = 1, /* this pad is for a cloned CV */
- padnew_SAVE = 2, /* save old globals */
- padnew_SAVESUB = 4 /* also save extra stuff for start of sub */
-} padnew_flags;
+#define padnew_CLONE 1 /* this pad is for a cloned CV */
+#define padnew_SAVE 2 /* save old globals */
+#define padnew_SAVESUB 4 /* also save extra stuff for start of sub */
/* values for the pad_tidy() function */
Set the current pad to be pad C<n> in the padlist, saving
the previous current pad.
+=for apidoc m|void|PAD_SET_CUR_NOSAVE |PADLIST padlist|I32 n
+like PAD_SET_CUR, but without the save
+
=for apidoc m|void|PAD_SAVE_SETNULLPAD
Save the current pad then set it to null.
? AvARRAY((AV*)(AvARRAY(padlist)[1]))[po] : Nullsv;
-#define PAD_SET_CUR(padlist,n) \
- SAVECOMPPAD(); \
+#define PAD_SET_CUR_NOSAVE(padlist,n) \
PL_comppad = (PAD*) (AvARRAY(padlist)[n]); \
PL_curpad = AvARRAY(PL_comppad); \
DEBUG_Xv(PerlIO_printf(Perl_debug_log, \
PTR2UV(PL_comppad), PTR2UV(PL_curpad), (int)(n)));
+#define PAD_SET_CUR(padlist,n) \
+ SAVECOMPPAD(); \
+ PAD_SET_CUR_NOSAVE(padlist,n);
+
+
#define PAD_SAVE_SETNULLPAD() SAVECOMPPAD(); \
PL_comppad = Null(PAD*); PL_curpad = Null(SV**); \
DEBUG_Xv(PerlIO_printf(Perl_debug_log, "Pad set_null\n"));
else \
(dstpad) = av_dup_inc((srcpad), param);
-/* note - we set comp/curpad to null rather than duping - otherwise
- * we may dup a pad but not the whole padlist, and be left with
- * leaked pad. We assume that a sub will get called very soon hereafter
- * and comp/curpad will get set to something sensible. DAPM 16-Oct02 */
-/* XXX DAPM -does the same logic appply to comppad_name ? */
+/* NB - we set PL_comppad to null unless it points at a value that
+ * has already been dup'ed, ie it points to part of an active padlist.
+ * Otherwise PL_comppad ends up being a leaked scalar in code like
+ * the following:
+ * threads->create(sub { threads->create(sub {...} ) } );
+ * where the second thread dups the outer sub's comppad but not the
+ * sub's CV or padlist. */
#define PAD_CLONE_VARS(proto_perl, param) \
- PL_comppad = Null(PAD*); \
- PL_curpad = Null(SV **); \
+ PL_comppad = ptr_table_fetch(PL_ptr_table, proto_perl->Icomppad); \
+ PL_curpad = PL_comppad ? AvARRAY(PL_comppad) : Null(SV**); \
PL_comppad_name = av_dup(proto_perl->Icomppad_name, param); \
PL_comppad_name_fill = proto_perl->Icomppad_name_fill; \
PL_comppad_name_floor = proto_perl->Icomppad_name_floor; \