Object Method Shorthand

[xjamundx]

Convert object methods into shorthand

{
    x: function() { }
}

// into
{
    x() {}
}

[jsnmoon]

For anyone interested in picking up this task, it'll be important to distinguish the difference between the two cases.

Case A: (anonymous function)

{
  x: function() { ... }
}

Case B: (named function)

{
  x: function someNamedFunction() { ... }
}

Automatically converting the unnamed function shouldn't cause any issues, but converting the named function can be problematic if the named function involves recursion or event binding/unbinding - a good example is documented here.

If we're ok letting this be a potentially breaking transformation, I think we should go ahead and convert both cases. If we want this to be a safe transformation, I think we should only transform case A.

As an aside, it might be nice to have the ability to an allowUnsafe flag for this transform; we can support both transformations while only enabling the safe transformation by default.