https://bugs.freedesktop.org/show_bug.cgi?id=51324

Stephan Bergmann <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #30 from Stephan Bergmann <[email protected]> 2012-08-16 07:31:26 
UTC ---
(In reply to comment #26)
> Reopening, finally found the root cause of this it seems and LibreOffice is 
> not
> really innocent.

Just for the record, it more looks like a problem of boost than of LibreOffice
to me (though the commit that happens to fix it is fine in and of itself
anyway, of course); quoting recent #libreoffice-dev:

Aug 07 11:56:18 <Sweetshark> caolan: could you please review
https://gerrit.libreoffice.org/#/c/373/ for libreoffice-3-6 ?
Aug 07 11:57:28 <sberg> Sweetshark, but aCandidates.first/second are not used
after erase, so the original code should be fine?
Aug 07 12:01:21 <Sweetshark> sberg: afaik the iterator are not guaranteed to be
stable _inside_ an erase (at least a few stl pages warned about that).
Aug 07 12:02:15 <sberg> Sweetshark, and "Remove the links [plural!] from the
given window" suggests that there can indeed be multiple entries for pWindow
(after all, its an unordered_multimap)
Aug 07 12:02:47 <sberg> Sweetshark, "not guaranteed to be stable": that would
render erase(iterator,iterator) completely useless
Aug 07 12:07:38 <Sweetshark> sberg: fact is: without that I crash after ~10
iterations, with the change it crashes after >100 iterations on a different
issue here. So either we are doing something illegal (which -- as you say is
unlikely), or boost-1.49/gcc4.7 is broken wrt that.
Aug 07 12:15:04 <sberg> Sweetshark, or, only removing a single entry per
pWindow instead of all of them happens to mask some other error
Aug 07 12:17:05 <Sweetshark> sberg: huh? according to boost docs erase(key&)
also kills _all_ pWindows
Aug 07 12:18:39 <sberg> Sweetshark, ah, right;  odd, then
Aug 07 12:20:19 <Sweetshark> sberg: note I also replaced some of the std::lists
with std::deques before to evade ABI breakage. however that did not fix the
issue.

-- 
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to