-
Notifications
You must be signed in to change notification settings - Fork 119
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
Morecmek #1450
Morecmek #1450
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1450 +/- ##
==========================================
- Coverage 17.2% 17.19% -0.01%
==========================================
Files 164 164
Lines 17104 17103 -1
==========================================
- Hits 2942 2941 -1
Misses 14162 14162
Continue to review full report at Codecov.
|
|
||
// IDE_EDITABLEs added before compile | ||
virtual int compile_parseAndLink(EnigmaStruct*,parsed_script*[], vector<parsed_script*>& tlines, const std::set<std::string>& script_names) = 0; | ||
virtual int compile_parseSecondary(map<int,parsed_object*>&,parsed_script*[],int scrcount, vector<parsed_script*>& tlines, map<int,parsed_room*>&,parsed_object*, const std::set<std::string>&) = 0; | ||
virtual int compile_parseAndLink(EnigmaStruct*, vector<parsed_script*>&, vector<parsed_script*>& tlines, const std::set<std::string>& script_names) = 0; |
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.
Prefer const&
so it's clear nothing is being mutated (the stuff behind the pointers may be, but that's fine).
SOURCE_DIR "${CMAKE_SOURCE_DIR}/CompilerSource" | ||
BINARY_DIR "${CMAKE_SOURCE_DIR}/build/compiler" | ||
CMAKE_ARGS ${CL_ARGS} | ||
) |
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're missing a newline here.
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.
this pr should be closed. idk y some idiot reopened it
Closing in favor of #1642 which was recreated from this. The branch can be deleted some period of time after this has been closed. |
No description provided.