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

[dev.icinga.com #2773] Dyn: Optimize ObjectMap::RemoveObject #169

Closed
icinga-migration opened this issue Jul 2, 2012 · 3 comments
Closed
Labels
bug Something isn't working
Milestone

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/2773

Created by gbeutner on 2012-07-02 11:44:54 +00:00

Assignee: (none)
Status: Resolved (closed on 2012-07-27 18:58:07 +00:00)
Target Version: Milestone 1
Last Update: 2013-09-23 15:23:33 +00:00 (in Redmine)


The for loop is O (n²):

90 for (Iterator i = range.first; i != range.second; i**) {
91 if (i->second == object) {
92 m_Objects.erase(i);
93 break;
94 }
95 }

@icinga-migration
Copy link
Author

Updated by gbeutner on 2012-07-02 14:23:08 +00:00

  • Priority changed from Normal to Low

@icinga-migration
Copy link
Author

Updated by gbeutner on 2012-07-27 18:58:07 +00:00

  • Status changed from New to Resolved

ObjectMap/ObjectSet don't exist anymore.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2013-09-23 15:23:33 +00:00

  • Category set to libbase

@icinga-migration icinga-migration added Low bug Something isn't working labels Jan 17, 2017
@icinga-migration icinga-migration added this to the Milestone 1 milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant