Add examples to ol.coordinate functions

[marcjansen]

Title says it all.

[tschaub]

I'd be tempted to skip the return here (though your comment below clarifies, it risks misinterpretation since it is named otherCoord).

var coord = [-180, 45];
ol.coordinate.add(coord, [360, -90]);
// coord is now [180, -45]

[tschaub]

Nice @marcjansen! One comment on ol.coordinate.add that you can take or leave (it would be more consistent with your other examples not to use the return). It might be nice to update the title to read "Add examples to ol.coordinate functions" just so the changelog is more specific.

In any case, merge when ready.

[marcjansen]

Thanks for the review Tim. I'll change the mentioned parts and then merge.

[marcjansen]

Is there any chance that e81e030 and d0efa9e can be removed from this branch without destroying the content?

Is rebasing a usable otion here? I don't want to mess master's commit history.

[ahocevar]

@marcjansen You can use git rebase --interactive cd84d76. Then, in the text editor that pops up, remove the two lines with the merge commits. Caveat: I have removed commits with this, but I haven't tried it with merge commits, so triple-check the result before force-pushing.

[marcjansen]

@ahocevar git rebase --interactive cd84d76 gives me

Which will affect way more commit then this needs. I think I will simply create a new branch with the changes applied via a .diff

[marcjansen]

That worked fine, I'll merge once Travis is done.