[perl #46167] [TODO] Replace temporary hack for method cache invalidation with decent implementation

[James Keenan via RT]

On Sat Oct 06 05:07:17 2007, pcoch wrote:
> In src/global.c:Parrot_store_global_k() there is the todo item:
> 
> * XXX - temporary hack to notice when key is actually a string, so that
> * the legacy logic for invalidating method cache will be called; this
> * is not good enough but it avoids regressions for now

I can't locate 'store_global_k' in src/global.c anymore.

640:=item C<void Parrot_store_global_n(PARROT_INTERP, PMC *ns, STRING
*globalname,
651:Parrot_store_global_n(PARROT_INTERP, ARGIN_NULLOK(PMC *ns),
654:    ASSERT_ARGS(Parrot_store_global_n)
657:        Parrot_io_printf(interp, "store_global name '%Ss'\n",
globalname);
668:=item C<void Parrot_store_global_s(PARROT_INTERP, STRING *str_key,
STRING
680:Parrot_store_global_s(PARROT_INTERP, ARGIN_NULLOK(STRING *str_key),
683:    ASSERT_ARGS(Parrot_store_global_s)
688:    Parrot_store_global_n(interp, ns, globalname, val);
886:        Parrot_store_global_n(interp, ns, ns_entry_name, sub_pmc);

There are 4 other RTs still referenced in that file, but not this one:

254:    /* RT #46157 - stop depending on typed namespace */
258:    /* RT #46159 - match HLL of enclosing namespace? */
544:         * RT #46163 - we should be able to use 'get_pmc_keyed' here,
690:    /* RT #46169 - method cache invalidation should be a namespace
function */
759:    /* RT #46171 - walk up the scopes!  duh!! */

So I'm closing this ticket.

Thank you very much.
kid51