Add TODO

phetsims · Sep 13, 2019 · 42e091a · 42e091a
1 parent 14114d4
commit 42e091a
Showing 1 changed file with 2 additions and 0 deletions.
diff --git a/js/ObservableArray.js b/js/ObservableArray.js
@@ -227,6 +227,8 @@ define( function( require ) {
      * @public
      */
     pop: function() {
+
+      // TODO: should we really pop if the length was 0?  Or maybe this is OK?
       var item = this._array.pop();
       if ( item !== undefined ) {
         this.lengthProperty.set( this._array.length );