-
Notifications
You must be signed in to change notification settings - Fork 89
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
Polish and document the generator's code #25
Comments
Ok, fine, small-grained issues have higher chances of being closed soon ;-) |
yes, sort of ;) |
One related thing: I noticed that after the merge the compilation is no longer clean with -Wall -pedantic. Also, I once fixed the old generator such that -Wextra works (non-CUDA, since nvcc produces tons of warnings here) - could you please fix this? |
done :) 2013/5/19 Karl Rupp notifications@github.com
|
The generator is merged in the master branch, but the code can still be polished (some base classes to introduce, dirty code to remove ...) and is yet to be documented.
The text was updated successfully, but these errors were encountered: