[perl #60626] [DEPRECATED] Old-style MMD functions

Allison Randal allison at parrot.org
Sun Nov 23 19:39:09 UTC 2008


Will Coleda via RT wrote:
> 
> * Parrot_mmd_add_function
>   - src/inter_create.c //make_interpreter

Delete that line from src/inter_create.c. Also delete the line before 
which initializes 'interp->binop_mmd_funcs' to NULL.

These two lines are initializing the storage for the old MMD subsystem, 
which is never used anymore.

Note that this implies deprecating all functions that refer to 
'interp->binop_mmd_funcs'. (It looks like those are all caught in this 
list, just double-check to make sure. Ah, I don't see 'dump_mmd' in this 
list, it can also be removed, as it doesn't do anything anymore.)

> * mmd_cvt_to_types
>   - src/multidispatch.c //Parrot_mmd_get_cached_multi_sig

'Parrot_mmd_get_cached_multi_sig' is only called by 'mmd_distance', 
which is only called by 'Parrot_mmd_sort_candidates'. 
'Parrot_mmd_sort_candidates' is called all over the place (I 
standardized on that when I did the MMD cleanup).

Let's see what chromatic's MMD sort cleanup does. If it eliminates the 
call to 'mmd_distance' then problem solved. If not, then let's break the 
deprecation of 'mmd_cvt_to_types' out into a separate ticket for 
refactoring the unholy trio of 'mmd_cvt_to_types', 
'Parrot_mmd_get_cached_multi_sig', and 'mmd_distance'.

Allison


More information about the parrot-dev mailing list