-
Notifications
You must be signed in to change notification settings - Fork 27
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
Unexpected error message when using '-withReflectionCode'. #624
Comments
Note that |
Thanks to point this out. I agree that this behaviour is misleading. I think, we can move the checking of the command lines arguments from the cross language checking phase to the processing phase. This will avoid such failures. We will do it within the current milestone. The cross language checking phase has been introduced to find out that something will be clashed in the corresponding language even if the language is currently not used. Regarding |
Moved to the backlog due to more urgent issues. |
Moved from the backlog to the current milestone again because it turned out that we will need this for the new C++17 emitter. |
We will just move the checking of the command lines arguments from the cross language checking phase to the processing phase in C++. No other extensions suffer from this problem. |
Describe the bug
When I call
zserio myschema.zs -withReflectionCode -python zs_gen_api
, I get:But I haven't used the C++(11) support of zserio.
Expected behavior
zserio should not produce error messages for target languages I don't use. In addition
withReflectionCode
should just activatewithTypeInfo
if it is necessary. Imo there is nothing wrong with saying 'reflection covers type infos too.'.The text was updated successfully, but these errors were encountered: