GNU
/
easejs
1
0
Fork 0
Code Releases Activity

Rearranged util.overrideMethod() arguments to match methodOverride propCopy() action (to avoid unnecessary mistakes and confusion)

closure/master
Mike Gerwitz 2010-12-19 00:12:41 -05:00
parent 600e389b40
commit 1ff9408885
2 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
lib/class.js
View File

@ -126,7 +126,7 @@ var extend = ( function( extending )
methodOverride: function( name, pre, func )
{
return util.overrideMethod(
pre, func, name, result_data.abstractMethods
name, pre, func, result_data.abstractMethods
);
},
} );

6
lib/util.js
View File

@ -262,7 +262,7 @@ exports.propCopy = function( props, dest, actions )
var override_func = use_or( actions.methodOverride,
function( name, pre, func )
{
return exports.overrideMethod( pre, func, name );
return exports.overrideMethod( name, pre, func );
}
);
@ -341,15 +341,15 @@ exports.isAbstractMethod = function( func )
*
* The given method must be a function or an exception will be thrown.
*
* @param {string} name method name
* @param {Function} super_method method to override
* @param {Function} new_method method to override with
* @param {string} name method name
* @param {Array.<string>=} abstract_methods list of abstract methods
*
* @return {Function} overridden method
*/
exports.overrideMethod = function(
super_method, new_method, name, abstract_methods
name, super_method, new_method, abstract_methods
)
{
abstract_methods = abstract_methods || [];