Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

C++ port #257

Open
5 tasks
danpaz opened this issue Jul 25, 2018 · 2 comments
Open
5 tasks

C++ port #257

danpaz opened this issue Jul 25, 2018 · 2 comments

Comments

@danpaz
Copy link
Contributor

danpaz commented Jul 25, 2018

The logic in this module is written in javascript, but the translation strings and transifex integration are valuable features that are not necessarily tied to javascript. It would be great if we could use osrm-text-instructions in a native environment for example. A C++ port of this project would make this possible. A similar port of osrm-ti exists for iOS/macOS: https://github.com/Project-OSRM/osrm-text-instructions.swift/.

Approach

  • rewrite the main functions in c++
    • compile
    • tokenize
    • more as needed
  • write node bindings so we can use the existing js tests
@yuryleb
Copy link
Contributor

yuryleb commented Jul 25, 2018

I hope this will be started as separate repository and this one will continue to follow KISS? 😉

Actually I don't know who may need such "universal" C++ port (then why not C?) at all. Each product has its own resources format, why don't simple take original (and well-tested) JSONs and convert them as Swift port does? What's the problem to translate index.js code to your favorite language, it's simple and well- (ok, almost 😉) -commented (although this seems to be the problem for PHP developers 😟)?

@danpaz
Copy link
Contributor Author

danpaz commented Jul 25, 2018

@yuryleb I think this will probably become a separate repo similar to the swift one, but for now I'm working on a branch in this project to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants