Changeset c05547 in git for Singular/iparith.cc

Timestamp:

Aug 16, 2019, 3:41:57 PM (5 years ago)

Author:

Hans Schoenemann <hannes@…>

Branches:

(u'spielwiese', 'fe61d9c35bf7c61f2b6cbf1b56e25e2f08d536cc')

Children:

0a374dfc252034170c772ac44b6740e217190fbb0f194b4db842ab87efd9f6447b9793f2d687bceb309b8ae0c68d542d1d8a75fa675c060e90e18440

Parents:

c8b65d77e75cfce24c257822df19be0d1113e110a631cdc0aa559cc581e085981696acaa3b7216c1

git-author:

Hans Schoenemann <hannes@mathematik.uni-kl.de>2019-08-16 15:41:57+02:00

git-committer:

GitHub <noreply@github.com>2019-08-16 15:41:57+02:00

Message:

Merge pull request #950 from tthsqe12/no_length_in_division

opt: skip length calculation for division

File:

• Singular/iparith.cc (modified) (1 diff)

Singular/iparith.cc

@@ -3963 +3963 @@
 
 /// Return the denominator of the input number
-/// NOTE: the input number is normalized as a side effect
 static BOOLEAN jjDENOMINATOR(leftv res, leftv v)
 {
-  number n = reinterpret_cast<number>(v->Data());
+  number n = reinterpret_cast<number>(v->CopyD());
   res->data = reinterpret_cast<void*>(n_GetDenom(n, currRing->cf));
+  n_Delete(&n,currRing);
   return FALSE;
 }
 
 /// Return the numerator of the input number
-/// NOTE: the input number is normalized as a side effect
 static BOOLEAN jjNUMERATOR(leftv res, leftv v)
 {
-  number n = reinterpret_cast<number>(v->Data());
+  number n = reinterpret_cast<number>(v->CopyD());
   res->data = reinterpret_cast<void*>(n_GetNumerator(n, currRing->cf));
+  n_Delete(&n,currRing);
   return FALSE;
 }