Since several days a lot of nightly build test cases fail: _39e is not defined

[Paul Christopher]

Something seems to be broken in the build versions of Dojo (since a week or so?).

Check e.g.

​http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/form/test_Select.html --> _39e is not defined

​http://archive.dojotoolkit.org/nightly/dojotoolkit/dijit/tests/form/test_FilteringSelect.html --> Click on the arrow of the first drop down list so as to open it --> _39e is not defined

I have encountered this error a week ago when playing around with the new calendar widget, see ​https://github.com/damiengarbarino/dojo-calendar/issues/15.

With the calendar widget, the error fades away when I replace the compressed version by the uncompressed version of dojo.js.

[ben hockey]

this is a manifestation of a longstanding shrinksafe bug (#11673). assigning to mark wubben to fix. mark, don't use named functions - it's a problem with shrinksafe. ie - don't do this:

return lang.extend(function Deferred() {
  ...

  this.then = function () {
    ...
    listener.deferred = new Deferred();  // this reference will be minified (to _39e in this case) but the named function stays as Deferred
  };

});

[Mark Wubben]

In [28351]:

Remove use of named functions, refs #15179 !strict.

Linting fixes, refs #14615 !strict.

[Mark Wubben]

Uhm, OK. Better now?

[ben hockey]

i don't see the issue now. seems to be fixed. thanks.