]> www.ginac.de Git - ginac.git/blobdiff - ginac/ncmul.cpp
[BUGFIX] Fix crash in parser.
[ginac.git] / ginac / ncmul.cpp
index b37904bb76bd2dde96e48e8fa6bc6c5398b48a1a..5f64851a5920e52dd5a8fd7dd3e6540e9fb2c1a3 100644 (file)
@@ -3,7 +3,7 @@
  *  Implementation of GiNaC's non-commutative products of expressions. */
 
 /*
- *  GiNaC Copyright (C) 1999-2015 Johannes Gutenberg University Mainz, Germany
+ *  GiNaC Copyright (C) 1999-2024 Johannes Gutenberg University Mainz, Germany
  *
  *  This program is free software; you can redistribute it and/or modify
  *  it under the terms of the GNU General Public License as published by
@@ -57,30 +57,30 @@ ncmul::ncmul()
 
 // public
 
-ncmul::ncmul(const ex & lh, const ex & rh) : inherited(lh,rh)
+ncmul::ncmul(const ex & lh, const ex & rh) : inherited{lh,rh}
 {
 }
 
-ncmul::ncmul(const ex & f1, const ex & f2, const ex & f3) : inherited(f1,f2,f3)
+ncmul::ncmul(const ex & f1, const ex & f2, const ex & f3) : inherited{f1,f2,f3}
 {
 }
 
 ncmul::ncmul(const ex & f1, const ex & f2, const ex & f3,
-             const ex & f4) : inherited(f1,f2,f3,f4)
+             const ex & f4) : inherited{f1,f2,f3,f4}
 {
 }
 
 ncmul::ncmul(const ex & f1, const ex & f2, const ex & f3,
-             const ex & f4, const ex & f5) : inherited(f1,f2,f3,f4,f5)
+             const ex & f4, const ex & f5) : inherited{f1,f2,f3,f4,f5}
 {
 }
 
 ncmul::ncmul(const ex & f1, const ex & f2, const ex & f3,
-             const ex & f4, const ex & f5, const ex & f6) : inherited(f1,f2,f3,f4,f5,f6)
+             const ex & f4, const ex & f5, const ex & f6) : inherited{f1,f2,f3,f4,f5,f6}
 {
 }
 
-ncmul::ncmul(const exvector & v, bool discardable) : inherited(v,discardable)
+ncmul::ncmul(const exvector & v) : inherited(v)
 {
 }
 
@@ -146,8 +146,7 @@ ex ncmul::expand(unsigned options) const
        // If there are no sums, we are done
        if (number_of_adds == 0) {
                if (!v.empty())
-                       return (new ncmul(std::move(v)))->
-                               setflag(status_flags::dynallocated | (options == 0 ? status_flags::expanded : 0));
+                       return dynallocate<ncmul>(std::move(v)).setflag(options == 0 ? status_flags::expanded : 0);
                else
                        return *this;
        }
@@ -179,8 +178,7 @@ ex ncmul::expand(unsigned options) const
                        term[positions_of_adds[i]] = rename_dummy_indices_uniquely(va, expanded_seq[positions_of_adds[i]].op(k[i]), true);
                }
 
-               distrseq.push_back((new ncmul(term, true))->
-                                   setflag(status_flags::dynallocated | (options == 0 ? status_flags::expanded : 0)));
+               distrseq.push_back(dynallocate<ncmul>(std::move(term)).setflag(options == 0 ? status_flags::expanded : 0));
 
                // increment k[]
                int l = number_of_adds-1;
@@ -192,8 +190,7 @@ ex ncmul::expand(unsigned options) const
                        break;
        }
 
-       return (new add(distrseq))->
-               setflag(status_flags::dynallocated | (options == 0 ? status_flags::expanded : 0));
+       return dynallocate<add>(distrseq).setflag(options == 0 ? status_flags::expanded : 0);
 }
 
 int ncmul::degree(const ex & s) const
@@ -233,7 +230,7 @@ ex ncmul::coeff(const ex & s, int n) const
                // if a non-zero power of s is found, the resulting product will be 0
                for (auto & it : seq)
                        coeffseq.push_back(it.coeff(s,n));
-               return (new ncmul(std::move(coeffseq),1))->setflag(status_flags::dynallocated);
+               return dynallocate<ncmul>(std::move(coeffseq));
        }
                 
        bool coeff_found = false;
@@ -248,7 +245,7 @@ ex ncmul::coeff(const ex & s, int n) const
        }
 
        if (coeff_found)
-               return (new ncmul(std::move(coeffseq), 1))->setflag(status_flags::dynallocated);
+               return dynallocate<ncmul>(std::move(coeffseq));
        
        return _ex0;
 }
@@ -288,9 +285,8 @@ typedef std::vector<exvector> exvectorvector;
  *  - ncmul(...,c1,...,c2,...) -> *(c1,c2,ncmul(...))  (pull out commutative elements)
  *  - ncmul(x1,y1,x2,y2) -> *(ncmul(x1,x2),ncmul(y1,y2))  (collect elements of same type)
  *  - ncmul(x1,x2,x3,...) -> x::eval_ncmul(x1,x2,x3,...)
- *
- *  @param level cut-off in recursive evaluation */
-ex ncmul::eval(int level) const
+ */
+ex ncmul::eval() const
 {
        // The following additional rule would be nice, but produces a recursion,
        // which must be trapped by introducing a flag that the sub-ncmuls()
@@ -299,22 +295,20 @@ ex ncmul::eval(int level) const
        //                      ncmul(ncmul(x1,x2,...),X,ncmul(y1,y2,...)
        //                      (X noncommutative_composite)
 
-       if ((level==1) && (flags & status_flags::evaluated)) {
+       if (flags & status_flags::evaluated) {
                return *this;
        }
 
-       exvector evaledseq=evalchildren(level);
-
        // ncmul(...,*(x1,x2),...,ncmul(x3,x4),...) ->
        //     ncmul(...,x1,x2,...,x3,x4,...)  (associativity)
        size_t factors = 0;
-       for (auto & it : evaledseq)
+       for (auto & it : seq)
                factors += count_factors(it);
        
        exvector assocseq;
        assocseq.reserve(factors);
-       make_flat_inserter mf(evaledseq, true);
-       for (auto & it : evaledseq) {
+       make_flat_inserter mf(seq, true);
+       for (auto & it : seq) {
                ex factor = mf.handle_factor(it, 1);
                append_factors(assocseq, factor);
        }
@@ -364,8 +358,8 @@ ex ncmul::eval(int level) const
                        else
                                noncommutativeseq.push_back(assocseq[i]);
                }
-               commutativeseq.push_back((new ncmul(noncommutativeseq,1))->setflag(status_flags::dynallocated));
-               return (new mul(commutativeseq))->setflag(status_flags::dynallocated);
+               commutativeseq.push_back(dynallocate<ncmul>(std::move(noncommutativeseq)));
+               return dynallocate<mul>(std::move(commutativeseq));
        }
                
        // ncmul(x1,y1,x2,y2) -> *(ncmul(x1,x2),ncmul(y1,y2))
@@ -419,13 +413,12 @@ ex ncmul::eval(int level) const
                exvector splitseq;
                splitseq.reserve(evv_num);
                for (i=0; i<evv_num; ++i)
-                       splitseq.push_back((new ncmul(evv[i]))->setflag(status_flags::dynallocated));
+                       splitseq.push_back(dynallocate<ncmul>(evv[i]));
                
-               return (new mul(splitseq))->setflag(status_flags::dynallocated);
+               return dynallocate<mul>(splitseq);
        }
        
-       return (new ncmul(assocseq))->setflag(status_flags::dynallocated |
-                                                                                 status_flags::evaluated);
+       return dynallocate<ncmul>(assocseq).setflag(status_flags::evaluated);
 }
 
 ex ncmul::evalm() const
@@ -451,17 +444,17 @@ ex ncmul::evalm() const
        }
 
 no_matrix:
-       return (new ncmul(std::move(s)))->setflag(status_flags::dynallocated);
+       return dynallocate<ncmul>(std::move(s));
 }
 
 ex ncmul::thiscontainer(const exvector & v) const
 {
-       return (new ncmul(v))->setflag(status_flags::dynallocated);
+       return dynallocate<ncmul>(v);
 }
 
 ex ncmul::thiscontainer(exvector && v) const
 {
-       return (new ncmul(std::move(v)))->setflag(status_flags::dynallocated);
+       return dynallocate<ncmul>(std::move(v));
 }
 
 ex ncmul::conjugate() const
@@ -480,7 +473,7 @@ ex ncmul::conjugate() const
                --i;
                ev.push_back(i->conjugate());
        }
-       return (new ncmul(std::move(ev), true))->setflag(status_flags::dynallocated).eval();
+       return dynallocate<ncmul>(std::move(ev));
 }
 
 ex ncmul::real_part() const
@@ -509,10 +502,10 @@ ex ncmul::derivative(const symbol & s) const
        for (size_t i=0; i<num; ++i) {
                ex e = seq[i].diff(s);
                e.swap(ncmulseq[i]);
-               addseq.push_back((new ncmul(ncmulseq))->setflag(status_flags::dynallocated));
+               addseq.push_back(dynallocate<ncmul>(ncmulseq));
                e.swap(ncmulseq[i]);
        }
-       return (new add(addseq))->setflag(status_flags::dynallocated);
+       return dynallocate<add>(addseq);
 }
 
 int ncmul::compare_same_type(const basic & other) const
@@ -528,7 +521,7 @@ unsigned ncmul::return_type() const
        bool all_commutative = true;
        exvector::const_iterator noncommutative_element; // point to first found nc element
 
-       exvector::const_iterator i = seq.begin(), end = seq.end();
+       auto i = seq.begin(), end = seq.end();
        while (i != end) {
                unsigned rt = i->return_type();
                if (rt == return_types::noncommutative_composite)
@@ -615,7 +608,7 @@ const exvector & ncmul::get_factors() const
 
 ex reeval_ncmul(const exvector & v)
 {
-       return (new ncmul(v))->setflag(status_flags::dynallocated);
+       return dynallocate<ncmul>(v);
 }
 
 ex hold_ncmul(const exvector & v)
@@ -625,8 +618,7 @@ ex hold_ncmul(const exvector & v)
        else if (v.size() == 1)
                return v[0];
        else
-               return (new ncmul(v))->setflag(status_flags::dynallocated |
-                                              status_flags::evaluated);
+               return dynallocate<ncmul>(v).setflag(status_flags::evaluated);
 }
 
 GINAC_BIND_UNARCHIVER(ncmul);