Nicholas Clark <[EMAIL PROTECTED]> wrote:
> Looking at the code in array.pmc's set_integer_same I see:
>=item C<void set_integer_same(PMC* value)>
> Sets the length of the array to the number of elements in C<*value>.
>=cut
> */
> void set_integer_same (PMC* value) {
> INTVAL size = DYNSELF.elements();
> list_set_length(INTERP, (List *) PMC_data(SELF) ,size);
> }
> Surely that DYNSELF should be value? As is, I think that this method is a
> no-op.
Yep. That's true. Fixed. BTW did you come along that by using this
function or just by reading the code?
I find less and less usage for set_*_same and I'm inclined to just
delete it alltogether.
> Nicholas Clark
leo
