On Sun, Apr 14, 2024 at 10:17:39AM +0800, Qian Yun wrote:
> OK, in attachment are 2 patch files, please review.

Just a little remark: '(int)' truncates the number.  For bounding
box slightly bigger is not a problem, but I am affraid that too
small may be problematic.  So I would rather use '(int)ceil(...)'
intead of '(int)(...)'.

Also, those changes are really single thing, so put them in a
single commit.

-- 
                              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 fricas-devel+unsubscr...@googlegroups.com.
To view this discussion on the web visit 
https://groups.google.com/d/msgid/fricas-devel/Zhv1PGTx3ro4RQ7U%40fricas.org.

Reply via email to