> 
> --000000000000f11724058ed06cd0
> Content-Type: text/plain; charset="UTF-8"
> 
> On 7/27/19 6:32 PM, Waldek Hebisch wrote:
> > oldk1331 wrote:
> >> The second patch uses "construct" to greatly simplify "*" in MRING.
> >
> > When M is ordered old code tries to minimize need for sorting.
> > It is hard to see if this really gains some speed but it
> > would be better to do some measurements and compare speed
> > of old code and your new one.
> 
> You are right, old code is about 4 times faster than mine
> for MRING(INT,FreeMonoid(INT)).  (after work around the
> compiler bug in 'concat!')
> 
> So I'll change for "not (M has OrderedMonoid)" only.

OK
> 
> >> The third patch does some cleanup for MRING.
> 
> What's your opinion on the third patch?

OK

-- 
                              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 view this discussion on the web visit 
https://groups.google.com/d/msgid/fricas-devel/E1hs55S-0006vn-C1%40hera.math.uni.wroc.pl.

Reply via email to