Suggestion: function(){} replace by Ext.emptyFn

[Animal]

There are three places where no-op implementations of functions are specified as



function(){}


These can be replaced by a reference to the singleton no-op function Ext.emptyFn.


UpdateManager
DisplayField
GridPanel

[evant]

Got it, I'll update these in SVN.

[Animal]

It could be that the other ocurrences could use it too.

Like the default, no-op constructor created by Ext.extend if you do not provide one.

I don't know if I like the idea of a load of classes all sharing a constructor, even if it is null. Perhaps Jack can decide whether that level of byte counting is worth it/safe.

[Animal]

Ah, obviously Ext.extend cannot use it, it must create a new function with a new prototype, that's the whole point!