Changeset 271730 in git

Timestamp:

Mar 11, 2010, 9:51:15 AM (14 years ago)

Author:

Frank Seelisch <seelisch@…>

Branches:

(u'spielwiese', 'fe61d9c35bf7c61f2b6cbf1b56e25e2f08d536cc')

Children:

8e9b2432dca37f7a38575552109279486869c89e

Parents:

d07546b396be43cc23fe3dc01eb109d7682c6492

Message:

changed previous bug fix (which did not work for some of the slimgb instances): fix is now active in the ring case, only

git-svn-id: file:///usr/local/Singular/svn/trunk@12631 2c84dea3-7e68-4137-9b89-c4e89433aadc

File:

• kernel/kbuckets.cc (modified) (3 diffs)

kernel/kbuckets.cc

@@ -585 +585 @@
 #ifdef USE_COEF_BUCKETS
       if (i<coef_start)
-      {
         bucket->buckets[i] = p_Mult_nn(bucket->buckets[i], n, r);
 #ifdef HAVE_RINGS
+        /* Frank Seelisch on March 11, 2010:
+           This looks a bit strange: The following "if" is indented
+           like the previous line of code. But coded as it is,
+           it should actually be two spaces less indented.
+           Question: Should the following "if" also only be
+           performed when "(i<coef_start)" is true?
+           For the time being, I leave it as it is. */
         if (rField_is_Ring(r) && !(rField_is_Domain(r))) {
           bucket->buckets_length[i] = pLength(bucket->buckets[i]);
@@ -593 +599 @@
         }
 #endif
-      }
       else
       if (bucket->coef[i]!=NULL)
@@ -1114 +1119 @@
     {
       kBucket_Mult_n(bucket, an);
+#ifdef HAVE_RINGS
       lm = p_Mult_nn(lm, an, bucket->bucket_ring);
+#endif
     }
     rn = an;