2006-01-18 Frank Ch. Eigler <fche@elastic.org>

	* translate.cxx (c_unparser_assignment::visit_arrayindex):
	Eliminate dummy assignments for "<<<" code.

diff --git a/ChangeLog b/ChangeLog
index 824d798fad1146db5d05507189baa34d83eef5de..b6d84edb3ffcf9f1d694cf58b0ff5db498c57ff5 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2006-01-18  Frank Ch. Eigler  <fche@elastic.org>
+
+       * translate.cxx (c_unparser_assignment::visit_arrayindex):
+       Eliminate dummy assignments for "<<<" code.
+
 2006-01-17  Josh Stone  <joshua.i.stone@intel.com>
 
        PR 2156

diff --git a/translate.cxx b/translate.cxx
index ba3836821727c4d1d484077e4ff4ee998aaa033a..4fb608db3cbf0528c80b549ea09cefb9812d97f4 100644 (file)
--- a/translate.cxx
+++ b/translate.cxx
@@ -3156,7 +3156,7 @@ c_unparser_assignment::visit_arrayindex (arrayindex *e)
       // There is an exception to the above form: if we're doign a <<< assigment to 
       // a statistic-valued map, there's a special form we follow:
       //
-      // ({ tmp0=(idx0); ... tmpN=(idxN); rvar=(rhs); lvar; res;
+      // ({ tmp0=(idx0); ... tmpN=(idxN); rvar=(rhs);
       //    *no need to* lock (array);
       //    _stp_map_add_stat (array, idx0...N, rvar);
       //    *no need to* unlock (array);
@@ -3177,9 +3177,10 @@ c_unparser_assignment::visit_arrayindex (arrayindex *e)
          o->newline() << "c->last_stmt = " << lex_cast_qstring(*e->tok) << ";";
          // NB: *no need to* varlock_w guard (*parent, mvar);
          o->newline() << mvar.add (idx, rvar) << ";";
-         // dummy assignments
-         o->newline() << lvar << " = " << rvar << ";";
-         o->newline() << res << " = " << rvar << ";";
+          res = rvar;
+         // no need for these dummy assignments
+         // o->newline() << lvar << " = " << rvar << ";";
+         // o->newline() << res << " = " << rvar << ";";
        }
       else
        { // block used to control varlock_w lifespan