Hey committers-

AbstractMatrix.java uses clone() in four places. The code for the math
operations does a clone(), which copies all of 'this' into the clone,
then replaces the copied data with multiplied/divided/added entries.

Please change AbstractMatrix.java to use like() instead of clone();
and of course change the math operations code to read from 'this'
instead of the matrix created by like(). Using like() makes read-only
matrix types possible, and of course it avoids copying all of 'this'
twice.

It's like 20 lines of code total, so not worth a JIRA.

Thanks!

-- 
Lance Norskog
[email protected]

Reply via email to