Skip to content
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

A new round of fixes #227

Closed
wants to merge 31 commits into from
Closed

Conversation

avarga
Copy link
Collaborator

@avarga avarga commented Jun 20, 2024

No description provided.

…oncopyable

There is no need to copy them in the code, and existing implementation
of copying was incomplete.

Removed incorrect and unused copy constructor and operator=.
Descendant class didn't overrides these.
Base class is already initializing the nodeId_ in the same stage.
… name

par("useAvgInterference") instead of the correct par("computeAvgInterference").
…ction of cComponent

Replace otherModule->registerSignal() with (this->)registerSignal() in modules.
Replace otherModule->registerSignal() with omnetpp::cComponent::registerSignal() in other classes.
The module already has the parameter "feedbackGeneratorType", getAncestorPar() found that.
…lity

The ip2nic module was overlapped a connection.
…le(): already exists in base BackgroundTrafficManager class
@avarga avarga closed this Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants