oldk1331 wrote:
> First, I found that default constructor, "construct", always
> copy its argument, that's inefficient if there are lots of
> elements. I propose a destructive constructor "construct!",
> that by potentially modify its argument, to achieve better
> performance.
1) In many cases constructed aggregate is has different structure
so there is no way to reuse argument of construct
2) In actual tests copy in 'constuct' was never a bottleneck,
so I would not worry to much.
>
> The second one is a minor issue, bracket already calls
> commaSeparate, and Stack can use the default implementation.
Applied (with simiar change in a few other places).
--
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.
