-
-
Notifications
You must be signed in to change notification settings - Fork 19.6k
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
Get rid of some compilation warnings. #7851
Get rid of some compilation warnings. #7851
Conversation
modules/gdscript/gd_editor.cpp
Outdated
@@ -2645,6 +2645,8 @@ Error GDScriptLanguage::lookup_code(const String& p_code, const String& p_symbol | |||
|
|||
switch(p.get_completion_type()) { | |||
|
|||
case GDParser::COMPLETION_GET_NODE: { | |||
} break; | |||
case GDParser::COMPLETION_NONE: { |
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.
You can do better here with:
case GDParser::COMPLETION_GET_NODE:
case GDParser::COMPLETION_NONE: {
} break;
main/tests/test_gdscript.cpp
Outdated
@@ -318,6 +318,7 @@ static void _parser_show_block(const GDParser::BlockNode *p_block,int p_indent) | |||
else | |||
_print_indent(p_indent,"return "); | |||
} break; | |||
case GDParser::ControlFlowNode::CF_MATCH: break; |
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.
That one is wrong, it should "unparse" the match
statement, instead of just covering the issue in a break. If you think it would be too complex for the scope of this PR, you should either leave a TODO
comment, or not change it at all.
Hi @shlomif, nice to see a fellow Mageia dev here ;) |
Thanks for the commentary @bojidar-bg ! I'll try to tend to it soon. |
Redone the commit based on the input in godotengine#7851 . Not all warnings were fixed but it's a start.
af83f10
to
0a2c387
Compare
OK , I redid the commit per @bojidar-bg 's comments. |
These appear by default on Mageia Linux v6. Not all compilation warnings
were fixed but it's a start.