graphhopper / map-matching

The map matching functionality is now located in the main repository https://github.com/graphhopper/graphhopper#map-matching
https://www.graphhopper.com/open-source/
Apache License 2.0
785 stars 274 forks source link

Clean up MapMatching.java #69

Closed stefanholder closed 8 years ago

stefanholder commented 8 years ago

Cleaned up MapMatching.java and moved code from offline_map_matching branch-1.0 to separate package.

karussell commented 8 years ago

As many changes are also regarding code formatting, would you mind to check this. Or probably we should apply the new code formatting to the whole project at a later point?

stefanholder commented 8 years ago

As many changes are also regarding code formatting, would you mind to check this. Or probably we should apply the new code formatting to the whole project at a later point?

I didn't realize before that the Graphhopper code formatting rules were so recent. However, I think that the tabs I accidentally inserted in #66 should definitely be converted to spaces because mixing tabs and spaces is really bad.

If you prefer I can undo the line break changes. However, I only changed MapMatching.java, which should cause no trouble with pending pull requests and you can still apply the new code formatting to the whole project later.

karussell commented 8 years ago

Thanks!

However, I only changed MapMatching.java, which should cause no trouble with pending pull requests and you can still apply the new code formatting to the whole project later.

yes, ok