Identifiers in function calls with an object are now processed

[Janther]

closes #680

Had to add Comments.sol to the list of tests that should not be formatted twice because of this:

// Original
purchaseData[0] = DomainPurchaseData({
  /* test */prices: _rootPrices,
  // test2
  subdomainMintingEnabled: /* test3 */_rootPrices.short > 0,
  allowSubdomainsToMint: true, // test4
  wasAllowedToSubdomainMintOnCreation: true
});

// 1st format
purchaseData[0] = DomainPurchaseData({
    /* test */
    prices: _rootPrices,
    // test2
    subdomainMintingEnabled: /* test3 */
    _rootPrices.short > 0,
    allowSubdomainsToMint: true, // test4
    wasAllowedToSubdomainMintOnCreation: true
});

// 2nd format
purchaseData[0] = DomainPurchaseData({
    /* test */
    prices: _rootPrices,
    // test2
    subdomainMintingEnabled: _rootPrices.short > 0, /* test3 */
    allowSubdomainsToMint: true, // test4
    wasAllowedToSubdomainMintOnCreation: true
});

As you see the second format places the comment /* test3 */ at the end of the whole line instead of between the identifier and the value.

It's something that will be reviewed but for now this is a great improvement.

@fvictorio We are no longer using the names attribute for this printer. Maybe it can be removed from the parser.