From: Nick Ing-Simmons Date: Fri, 1 Sep 2000 17:21:57 +0000 (+0000) Subject: Document UNTIE. Also tweak implementation to suppress the 'inner references' X-Git-Url: http://git.shadowcat.co.uk/gitweb/gitweb.cgi?a=commitdiff_plain;h=301e812593b886ce092a67093cee831022be6f82;p=p5sagit%2Fp5-mst-13.2.git Document UNTIE. Also tweak implementation to suppress the 'inner references' warning when UNTIE exists and instead pass the count of extra references to the UNTIE method. p4raw-id: //depot/perl@6981 --- diff --git a/pod/perltie.pod b/pod/perltie.pod index d08ed56..60df0cb 100644 --- a/pod/perltie.pod +++ b/pod/perltie.pod @@ -48,7 +48,7 @@ for you--you need to do that explicitly yourself. =head2 Tying Scalars A class implementing a tied scalar should define the following methods: -TIESCALAR, FETCH, STORE, and possibly DESTROY. +TIESCALAR, FETCH, STORE, and possibly UNTIE and/or DESTROY. Let's look at each in turn, using as an example a tie class for scalars that allows the user to do something like: @@ -157,6 +157,12 @@ argument--the new value the user is trying to assign. return $new_nicety; } +=item UNTIE this + +This method will be triggered when the C occurs. This can be useful +if the class needs to know when no further calls will be made. (Except DESTROY +of course.) See below for more details. + =item DESTROY this This method will be triggered when the tied variable needs to be destructed. @@ -180,7 +186,7 @@ TIESCALAR classes are certainly possible. =head2 Tying Arrays A class implementing a tied ordinary array should define the following -methods: TIEARRAY, FETCH, STORE, FETCHSIZE, STORESIZE and perhaps DESTROY. +methods: TIEARRAY, FETCH, STORE, FETCHSIZE, STORESIZE and perhaps UNTIE and/or DESTROY. FETCHSIZE and STORESIZE are used to provide C<$#array> and equivalent C access. @@ -192,7 +198,7 @@ base class to implement the first five of these in terms of the basic methods above. The default implementations of DELETE and EXISTS in B simply C. -In addition EXTEND will be called when perl would have pre-extended +In addition EXTEND will be called when perl would have pre-extended allocation in a real array. This means that tied arrays are now I. The example below needs @@ -260,10 +266,10 @@ index whose value we're trying to fetch. return $self->{ARRAY}[$idx]; } -If a negative array index is used to read from an array, the index +If a negative array index is used to read from an array, the index will be translated to a positive one internally by calling FETCHSIZE -before being passed to FETCH. - +before being passed to FETCH. + As you may have noticed, the name of the FETCH method (et al.) is the same for all accesses, even though the constructors differ in names (TIESCALAR vs TIEARRAY). While in theory you could have the same class servicing @@ -285,8 +291,12 @@ there. For example: } return $self->{ARRAY}[$idx] = $value; } - -Negative indexes are treated the same as with FETCH. + +Negative indexes are treated the same as with FETCH. + +=item UNTIE this + +Will be called when C happens. (See below.) =item DESTROY this @@ -316,8 +326,8 @@ the constructor. FETCH and STORE access the key and value pairs. EXISTS reports whether a key is present in the hash, and DELETE deletes one. CLEAR empties the hash by deleting all the key and value pairs. FIRSTKEY and NEXTKEY implement the keys() and each() functions to iterate over all -the keys. And DESTROY is called when the tied variable is garbage -collected. +the keys. UNTIE is called when C happens, and DESTROY is called when +the tied variable is garbage collected. If this seems like a lot, then feel free to inherit from merely the standard Tie::Hash module for most of your methods, redefining only the @@ -599,6 +609,10 @@ thing, but we'll have to go through the LIST field indirectly. return each %{ $self->{LIST} } } +=item UNTIE this + +This is called when C occurs. + =item DESTROY this This method is triggered when a tied hash is about to go out of @@ -629,7 +643,7 @@ This is partially implemented now. A class implementing a tied filehandle should define the following methods: TIEHANDLE, at least one of PRINT, PRINTF, WRITE, READLINE, GETC, -READ, and possibly CLOSE and DESTROY. The class can also provide: BINMODE, +READ, and possibly CLOSE, UNTIE and DESTROY. The class can also provide: BINMODE, OPEN, EOF, FILENO, SEEK, TELL - if the corresponding perl operators are used on the handle. @@ -718,6 +732,11 @@ function. sub CLOSE { print "CLOSE called.\n" } +=item UNTIE this + +As with the other types of ties, this method will be called when C happens. +It may be appropriate to "auto CLOSE" when this occurs. + =item DESTROY this As with the other types of ties, this method will be called when the @@ -855,7 +874,8 @@ closed. The reason there is no output is because the file buffers have not been flushed to disk. Now that you know what the problem is, what can you do to avoid it? -Well, the good old C<-w> flag will spot any instances where you call +Prior to the introduction of the optional UNTIE method the only way +was the good old C<-w> flag. Which will spot any instances where you call untie() and there are still valid references to the tied object. If the second script above this near the top C or was run with the C<-w> flag, Perl prints this @@ -870,6 +890,25 @@ called: undef $x; untie $fred; +Now that UNTIE exists the class designer can decide which parts of the +class functionality are really associated with C and which with +the object being destroyed. What makes sense for a given class depends +on whether the inner references are being kept so that non-tie-related +methods can be called on the object. But in most cases it probably makes +sense to move the functionality that would have been in DESTROY to the UNTIE +method. + +If the UNTIE method exists then the warning above does not occur. Instead the +UNTIE method is passed the count of "extra" references and can issue its own +warning if appropriate. e.g. to replicate the no UNTIE case this method can +be used: + + sub UNTIE + { + my ($obj,$count) = @_; + carp "untie attempted while $count inner references still exist" if $count; + } + =head1 SEE ALSO See L or L for some interesting tie() implementations. @@ -891,3 +930,6 @@ source code to MLDBM. Tom Christiansen TIEHANDLE by Sven Verdoolaege > and Doug MacEachern > + +UNTIE by Nick Ing-Simmons > + diff --git a/pp_sys.c b/pp_sys.c index 371c4a3..74011fb 100644 --- a/pp_sys.c +++ b/pp_sys.c @@ -29,7 +29,7 @@ * --jhi */ # ifdef __hpux__ /* There is a MAXINT coming from <- <- - * and another MAXINT from "perl.h" <- . */ + * and another MAXINT from "perl.h" <- . */ # undef MAXINT # endif # include @@ -40,8 +40,8 @@ # include #endif -#ifdef HAS_SYSCALL -#ifdef __cplusplus +#ifdef HAS_SYSCALL +#ifdef __cplusplus extern "C" int syscall(unsigned long,...); #endif #endif @@ -58,7 +58,7 @@ extern "C" int syscall(unsigned long,...); # include # if defined(USE_SOCKS) && defined(I_SOCKS) # include -# endif +# endif # ifdef I_NETDB # include # endif @@ -703,7 +703,7 @@ PP(pp_binmode) if (MAXARG > 1) discp = POPs; - gv = (GV*)POPs; + gv = (GV*)POPs; if (gv && (mg = SvTIED_mg((SV*)gv, 'q'))) { PUSHMARK(SP); @@ -722,7 +722,7 @@ PP(pp_binmode) if (!(io = GvIO(gv)) || !(fp = IoIFP(io))) RETPUSHUNDEF; - if (do_binmode(fp,IoTYPE(io),mode_from_discipline(discp))) + if (do_binmode(fp,IoTYPE(io),mode_from_discipline(discp))) RETPUSHYES; else RETPUSHUNDEF; @@ -769,7 +769,7 @@ PP(pp_tie) PUSHs(*MARK++); PUTBACK; call_method(methname, G_SCALAR); - } + } else { /* Not clear why we don't call call_method here too. * perhaps to get different error message ? @@ -777,7 +777,7 @@ PP(pp_tie) stash = gv_stashsv(*MARK, FALSE); if (!stash || !(gv = gv_fetchmethod(stash, methname))) { DIE(aTHX_ "Can't locate object method \"%s\" via package \"%s\"", - methname, SvPV(*MARK,n_a)); + methname, SvPV(*MARK,n_a)); } ENTER; PUSHSTACKi(PERLSI_MAGIC); @@ -813,22 +813,23 @@ PP(pp_untie) SV *obj = SvRV(mg->mg_obj); GV *gv; CV *cv = NULL; - if (ckWARN(WARN_UNTIE)) { - if (mg && SvREFCNT(obj) > 1) - Perl_warner(aTHX_ WARN_UNTIE, - "untie attempted while %"UVuf" inner references still exist", - (UV)SvREFCNT(obj) - 1 ) ; - } if ((gv = gv_fetchmethod_autoload(SvSTASH(obj), "UNTIE", FALSE)) && isGV(gv) && (cv = GvCV(gv))) { PUSHMARK(SP); XPUSHs(SvTIED_obj((SV*)gv, mg)); + XPUSHs(sv_2mortal(newSViv(SvREFCNT(obj)-1))); PUTBACK; ENTER; call_sv((SV *)cv, G_VOID); LEAVE; SPAGAIN; } + else if (ckWARN(WARN_UNTIE)) { + if (mg && SvREFCNT(obj) > 1) + Perl_warner(aTHX_ WARN_UNTIE, + "untie attempted while %"UVuf" inner references still exist", + (UV)SvREFCNT(obj) - 1 ) ; + } } sv_unmagic(sv, how); RETPUSHYES; @@ -901,7 +902,7 @@ PP(pp_dbmopen) } if (sv_isobject(TOPs)) { - sv_unmagic((SV *) hv, 'P'); + sv_unmagic((SV *) hv, 'P'); sv_magic((SV*)hv, TOPs, 'P', Nullch, 0); } LEAVE; @@ -1788,7 +1789,7 @@ PP(pp_eof) PP(pp_tell) { djSP; dTARGET; - GV *gv; + GV *gv; MAGIC *mg; if (MAXARG == 0) @@ -1890,7 +1891,7 @@ PP(pp_truncate) len = (Off_t)POPi; #endif /* Checking for length < 0 is problematic as the type might or - * might not be signed: if it is not, clever compilers will moan. */ + * might not be signed: if it is not, clever compilers will moan. */ /* XXX Configure probe for the signedness of the length type of *truncate() needed? XXX */ SETERRNO(0,0); #if defined(HAS_TRUNCATE) || defined(HAS_CHSIZE) || defined(F_FREESP) @@ -1904,7 +1905,7 @@ PP(pp_truncate) PerlIO_flush(IoIFP(GvIOp(tmpgv))); #ifdef HAS_TRUNCATE if (ftruncate(PerlIO_fileno(IoIFP(GvIOn(tmpgv))), len) < 0) -#else +#else if (my_chsize(PerlIO_fileno(IoIFP(GvIOn(tmpgv))), len) < 0) #endif result = 0; @@ -2005,7 +2006,7 @@ PP(pp_ioctl) retval = fcntl(PerlIO_fileno(IoIFP(io)), func, (int)s); #else retval = fcntl(PerlIO_fileno(IoIFP(io)), func, s); -#endif +#endif #else DIE(aTHX_ "fcntl is not implemented"); #endif @@ -2492,7 +2493,7 @@ PP(pp_getpeername) if (((struct sockaddr *)SvPVX(sv))->sa_family == AF_INET && !memcmp((char *)SvPVX(sv) + sizeof(u_short), nowhere, sizeof(u_short) + sizeof(struct in_addr))) { - goto nuts2; + goto nuts2; } } #endif @@ -2604,7 +2605,7 @@ PP(pp_stat) PUSHs(sv_2mortal(newSVuv(PL_statcache.st_uid))); # endif #endif -#if Gid_t_size > IVSIZE +#if Gid_t_size > IVSIZE PUSHs(sv_2mortal(newSVnv(PL_statcache.st_gid))); #else # if Gid_t_sign <= 0 @@ -3156,7 +3157,7 @@ PP(pp_fttext) break; } #ifdef EBCDIC - else if (!(isPRINT(*s) || isSPACE(*s))) + else if (!(isPRINT(*s) || isSPACE(*s))) odd++; #else else if (*s & 128) { @@ -3744,7 +3745,7 @@ PP(pp_fork) PP(pp_wait) { -#if (!defined(DOSISH) || defined(OS2) || defined(WIN32)) && !defined(MACOS_TRADITIONAL) +#if (!defined(DOSISH) || defined(OS2) || defined(WIN32)) && !defined(MACOS_TRADITIONAL) djSP; dTARGET; Pid_t childpid; int argflags; @@ -3765,7 +3766,7 @@ PP(pp_wait) PP(pp_waitpid) { -#if (!defined(DOSISH) || defined(OS2) || defined(WIN32)) && !defined(MACOS_TRADITIONAL) +#if (!defined(DOSISH) || defined(OS2) || defined(WIN32)) && !defined(MACOS_TRADITIONAL) djSP; dTARGET; Pid_t childpid; int optype; @@ -4559,7 +4560,7 @@ PP(pp_gprotoent) #if defined(HAS_GETPROTOBYNAME) || defined(HAS_GETPROTOBYNUMBER) || defined(HAS_GETPROTOENT) I32 which = PL_op->op_type; register char **elem; - register SV *sv; + register SV *sv; #ifndef HAS_GETPROTO_PROTOS /* XXX Do we need individual probes? */ struct protoent *PerlSock_getprotobyname(Netdb_name_t); struct protoent *PerlSock_getprotobynumber(int); @@ -4844,7 +4845,7 @@ PP(pp_gpwent) register SV *sv; STRLEN n_a; struct passwd *pwent = NULL; - /* + /* * We currently support only the SysV getsp* shadow password interface. * The interface is declared in and often one needs to link * with -lsecurity or some such. @@ -4885,7 +4886,7 @@ PP(pp_gpwent) * * Note that is already probed for, but currently * it is only included in special cases. - * + * * In Digital UNIX/Tru64 if using the getespw*() (which seems to be * be preferred interface, even though also the getprpw*() interface * is available) one needs to link with -lsecurity -ldb -laud -lm. @@ -5216,7 +5217,7 @@ PP(pp_syscall) a[i++] = SvIV(*MARK); else if (*MARK == &PL_sv_undef) a[i++] = 0; - else + else a[i++] = (unsigned long)SvPV_force(*MARK, n_a); if (i > 15) break; @@ -5284,7 +5285,7 @@ PP(pp_syscall) } #ifdef FCNTL_EMULATE_FLOCK - + /* XXX Emulate flock() with fcntl(). What's really needed is a good file locking module. */ @@ -5293,7 +5294,7 @@ static int fcntl_emulate_flock(int fd, int operation) { struct flock flock; - + switch (operation & ~LOCK_NB) { case LOCK_SH: flock.l_type = F_RDLCK; @@ -5310,7 +5311,7 @@ fcntl_emulate_flock(int fd, int operation) } flock.l_whence = SEEK_SET; flock.l_start = flock.l_len = (Off_t)0; - + return fcntl(fd, (operation & LOCK_NB) ? F_SETLK : F_SETLKW, &flock); }