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.

Sign up for GitHub

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

Jump to bottom

Refactor make_table #311

Merged

texodus merged 5 commits into master from parse_data_refactor

Nov 16, 2018

Contributor

sc1f commented Nov 14, 2018

Reduce the number of parameters to make_table in cpp
Add --source-map-base for emcc, should kick in when source map generation for wasm is fixed
Refactor out repetitive lines of code associated with make_table

sc1f and others added 5 commits

November 14, 2018 15:35


          reduce limit index logic to one function

42a507d


          add process_table to clean up make_table

091a2c1


          reduce parameter count on make_table

75e0fa3


          refactor out parse_data into separate file

9f9564b


          Further refactoring, moving make_table, _calculate_computed to C++

44ebf4e

texodus merged commit e725ea2 into master

texodus deleted the parse_data_refactor branch

November 16, 2018 01:07

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet