https://bugs.documentfoundation.org/show_bug.cgi?id=104686

Julien Nabet <[email protected]> changed:

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

--- Comment #12 from Julien Nabet <[email protected]> ---
Caolan: thought you might be interested in this one (since vcl part)

2 weird things:
1) I wonder why menues are empty
2) in code source, function MenuFloatingWindow::ImplScroll, I noticed this:
    604     if ( bScrollUp && bUp )
    605     {
    606         nFirstEntry = pMenu->ImplGetPrevVisible( nFirstEntry );
    607         SAL_WARN_IF( nFirstEntry == ITEMPOS_INVALID, "vcl", "Scroll?!"
);
    608 
    609         long nScrollEntryHeight = pMenu->GetItemList()->GetDataFromPos(
nFirstEntry )->aSz.Height();


but in our case:
    625     else if ( bScrollDown && !bUp )
    626     {
    627         long nScrollEntryHeight = pMenu->GetItemList()->GetDataFromPos(
nFirstEntry )->aSz.Height();
    628 
    629         nFirstEntry = pMenu->ImplGetNextVisible( nFirstEntry );
    630         SAL_WARN_IF( nFirstEntry == ITEMPOS_INVALID, "vcl", "Scroll?!"
);

Shouldn't we mimic the first example so at least we might (I'm not sure since I
don't reproduce this) avoid the crash?

see
http://opengrok.libreoffice.org/xref/core/vcl/source/window/menufloatingwindow.cxx#592

It's been like this since
https://cgit.freedesktop.org/libreoffice/core/commit/?id=a61080c9bc96ca82f8416708792eca4a56623036
(2001-08-21)

-- 
You are receiving this mail because:
You are the assignee for the bug.
_______________________________________________
Libreoffice-bugs mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/libreoffice-bugs

Reply via email to