plasm.js icon indicating copy to clipboard operation
plasm.js copied to clipboard

fun.STRUCT in plasm-fun.js

Open Rayzen opened this issue 13 years ago • 0 comments


/**
   * STRUCT
   * 
   * @param {Array} items
   * @return {plasm.Model}
   * @api public
   */

  fun.STRUCT = function (items) {
    var transformations = function (o) {return o;};
    var objects = [];

    temp = [];

    items.forEach(function (item) {
      if (!(item instanceof plasm.Model) && 
          !(item instanceof plasm.Struct)) {
        transformations = COMP2([transformations, item]);
      } else {
        temp.push(APPLY([transformations, item]).clone());
        objects.push(APPLY([transformations, item]));
      }
    });

    return new plasm.Struct(objects, p);
  };

What's the necessity of the temp array in this code? it is declared locally, not visible outside the function, it isn't returned and it isn't never used (except that it is filled with clones of the items)...

Rayzen avatar Sep 21 '12 09:09 Rayzen