millermedeiros/Hasher — pull request #51 by gertsonderby (2013-03-04 12:51:15+00:00)
Title: Correction of 'define' invocation

Defining a module with a name (a string as first argument to define()) causes problems when the directory structure is different from the name given (i.e. when the project using Hasher has it placed e.g. in `/3rdparty/hasher.js`). Defining it as an anonymous module (AMD) is problem-free. For that reason, it makes sense to remove the name from wrapper.js' define statement, as I have done here.

"You may encounter some define() calls that include a name for the module as the first argument to define()[...] These are normally generated by the optimization tool. [...] It is normally best to avoid coding in a name for the module and just let the optimization tool burn in the module names."
- http://requirejs.org/docs/api.html#modulename