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

use mcpp to preprocess sources before CppReverse #74

Open
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

jeremysalwen
Copy link

This branch is NOT complete as it never actually /uses/ the included code, but it includes all the code to actually do the preprocessing. All that is necessary is a call to j_preprocess in preprocess.h

The arguments are mostly self-explanatory.

fin is the input file
fout is passed by reference. The location of the output file will be placed there.
include_args is the list of arguments to the preprocessor (such as "-I/usr/local/include", etc)
working_dir is the working directory for the preprocessor
errstream is a pointer to a string where the errors the preprocessor outputs will be stored
output_includes is a boolean option. If false, the preprocessor will not output text from included files (but will process it for macros).

The return value is an error code (0 indicates no error).

What is needed is the gui code to feed these options to the function j_preprocess, and simply use the preprocessed file as input to the lexer instead of the original.

mcpp will continue outputting through recoverable errors (such as #error directives or missing #includes), but we could include an option to discard the output if any errors occur.

mcpp is also highly portable (although I fiddled with the configuration a bit, so it is possible that it will need some patches to maintain its portability in this stripped form).

I have tested this on 64 bit debian/linux.

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

Successfully merging this pull request may close these issues.

1 participant