-
Notifications
You must be signed in to change notification settings - Fork 592
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
Slice library cleanup, remove dead code and other minor fixes #1677
Conversation
DictionaryPtr p = dynamic_pointer_cast<Dictionary>(matches.front()); | ||
if(p) | ||
{ | ||
if(_unit->ignRedefs()) |
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.
ignRedefs was always false, I think it was used before by local Slice and scripting languages.
@@ -5163,11 +4747,6 @@ Slice::Const::Const(const ContainerPtr& container, const string& name, const Typ | |||
_value(value), | |||
_literal(literal) | |||
{ | |||
if(valueType == 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.
this seems like some leftover from debugging the compilers
{ | ||
ContainedPtr contained2 = dynamic_pointer_cast<Contained>(_type); | ||
return (contained2 && contained2 == contained); | ||
} |
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 code was wrong with the upgrade to shared_ptr, and still everything worked! noticed it is not used anymore.
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.
Looks good
A follow-up PR to #1674, removes unused code from Slice library