Code style changes in LinearRing::getCentroid() #387

Closed
wants to merge 4 commits into from

1 participant

@marcjansen
OpenLayers member

Mainly:

  • Not too many var declarations
  • line-length
  • single return statement

Also added an identity check when compairing to the number 0.

This is mostly cosmetic follow-up to #361.

Relevant tests continue to pass in IE 6, FF 11 and the ancient Chrome 11.

Please review.

marcjansen added some commits Apr 3, 2012
@marcjansen marcjansen Code style changes: Not too many var declarations,
line-length, and single return statement. Added identity
check when compairing to the number 0.
b55db7b
@marcjansen marcjansen Incorporate review by @pagameba. 98a2643
@marcjansen
OpenLayers member

@pagameba, I incorporated your suggestions, tests continue to pass in Chromium 18.

Would you be so kind to have another look?

@marcjansen marcjansen closed this Aug 26, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment