-
Notifications
You must be signed in to change notification settings - Fork 503
Add the formula for number of indexes per resolution #69
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
printf("};\n"); | ||
} | ||
|
||
int main(int argc, char* argv[]) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alternatively, you could change this to int main(void)
and ignore the arguments entirely.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we make that change, I think it should be made to all the filters. edit: by which I mean all the apps its relevant for.
printf("};\n"); | ||
} | ||
|
||
int main(int argc, char* argv[]) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1
Add the formula for number of indexes per resolution
This seems a little inefficient to put into its own file, but I'll put it there for the moment and later refactor the generators. Especially so the generators do not depend on the files whose contents they are partly generating (perhaps have a generated tables file, and a base tables/constants file it's based off of.)