oldk1331 wrote:
> 
> I see this change in openaxiom,
> https://github.com/GabrielDosReis/open-axiom/commit/67f8ca75d03aad7f041d8650ddca4d4cd0b71371
> 
> It seems reasonable, but instead of remove map/map!,
> we should move it into PrimitiveArray.
> (BTW, map/map! in PrimitiveArray is from IndexedAggregate,
> much slower than IARRAY.)
> 
> Should I make this change?

Inheriting from PrimitiveArray is dangerous: due to different
indexing all operations from PrimitiveArray are potentially
wrong for IARRAY.  Operations which are safe to inherit are
exceptional.  So the Open Axiom change looks undesirable.

Concerning map/map!: if we care about speed of them then we
need specialized version.  But I am not sure if we care.

-- 
                              Waldek Hebisch

-- 
You received this message because you are subscribed to the Google Groups 
"FriCAS - computer algebra system" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
Visit this group at https://groups.google.com/group/fricas-devel.
For more options, visit https://groups.google.com/d/optout.

Reply via email to