-
Notifications
You must be signed in to change notification settings - Fork 17
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
Merging yellow doors will now combine them to remove duplicate connec… #256
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.
Excellent work, as always!
Made a couple minor suggestions. After these, I think it's ready to merge.
DRODLib/OrbUtil.cpp
Outdated
|
||
#include "OrbUtil.h" | ||
|
||
void GetOrbConnections(CDbRoom& room, CCoordSet doorCoords, vector<COrbData*>& outOrbData) { |
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.
Performance optimization: Make doorCoords a const reference, i.e., const CCoordSet& doorCoords
. This avoids making a copy in calling the function, rather referencing the source object.
Same in next function.
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.
Still should make it a reference.
I.e., add '&' to avoid making a copy
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.
Sorry, I thought I did but I must've missed it
bc1c109
to
2cbaeb3
Compare
Try now! |
…tions and resolve conflicts by converting them to toggle
2cbaeb3
to
8cef282
Compare
Let's try again. |
…tions and resolve conflicts by converting them to toggle
Relevant thread