adobe/brackets — pull request #824 by mikechambers (2012-05-08 03:48:44+00:00)
Title: Key map optimizations

Refactoring of _normalizeKeyDescriptorString. Moved _isModifier definition out of _normalizeKeyDescriptorString and into scope of KeyMap require.js module.

Prior, _isModifier was defined each time _normalizeKeyDescriptorString wall called (18 time at startup). This lead to unnecessary code execution as well as garbage collection.

Added an additional argument to _isModifier, origDescriptor, which is used in a debug statement within the function.

_isModifier is now scoped to the KeyMap module, and thus will be in memory as long as the module is in memory.

Ignore the README.md commits. Git got in a weird state because i forked when brackets was private.

Note : you shoudl resquash when you pull it.