To comment on the following update, log in, then open the issue: http://www.openoffice.org/issues/show_bug.cgi?id=106576 Issue #|106576 Summary|wbutton.cxx: bad call to erase Component|Word processor Version|DEV300m63 Platform|All URL| OS/Version|Linux Status|UNCONFIRMED Status whiteboard| Keywords| Resolution| Issue type|DEFECT Priority|P3 Subcomponent|code Assigned to|writerneedsconfirm Reported by|dcb314
------- Additional comments from dcb...@openoffice.org Tue Nov 3 12:02:05 +0000 2009 ------- For source code file toolkit/source/layout/vcl/wbutton.cxx, function Remove, around line 500 is the following code for ( std::list< Window*>::iterator it = maAdvanced.begin(); it != maAdvanced.end(); it++ ) if ( *it == w ) { lst.erase( it ); return *it; } So local variable "it" iterates over maAdvanced, but it is used on variable "lst". This looks wrong. Maybe the loop should iterate over lst, not maAdvanced ? --------------------------------------------------------------------- Please do not reply to this automatically generated notification from Issue Tracker. Please log onto the website and enter your comments. http://qa.openoffice.org/issue_handling/project_issues.html#notification --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@sw.openoffice.org For additional commands, e-mail: issues-h...@sw.openoffice.org --------------------------------------------------------------------- To unsubscribe, e-mail: allbugs-unsubscr...@openoffice.org For additional commands, e-mail: allbugs-h...@openoffice.org