Hi Vadim,

M was crashing for me when I right-click on a message to e.g. move it,
or mark it read/unread. I managed to correct this problem with the
following patch, but I suspect that this may not be the correct way: it
seems we (I ?) mixed WXID_SEPARATOR (in the declaration of
popupMenuEntries) and WXMENU_SEPARATOR (in AppendToMenu), which do not
have the same values.

I'm using latest CVS versions of both M and wxWindows.


cvs diff -u src\gui\wxFolderView.cpp (in directory U:\src\M\)
Index: src/gui/wxFolderView.cpp
===================================================================
RCS file: /cvsroot/mahogany/M/src/gui/wxFolderView.cpp,v
retrieving revision 1.653
diff -u -r1.653 wxFolderView.cpp
--- src/gui/wxFolderView.cpp    16 Mar 2004 21:49:59 -0000      1.653
+++ src/gui/wxFolderView.cpp    17 Mar 2004 12:54:52 -0000
@@ -1750,7 +1750,10 @@
       for ( size_t n = 0; n < WXSIZEOF(popupMenuEntries); n++ )
       {
          int id = popupMenuEntries[n];
-         AppendToMenu(m_menu, id);
+         if (id == wxID_SEPARATOR)
+            m_menu->AppendSeparator();
+         else
+            AppendToMenu(m_menu, id);
       }
    }

-- 
Xavier Nodet
"They that can give up essential liberty to obtain a little temporary
safety deserve neither liberty nor safety." - Benjamin Franklin, 1759.

Attachment: pgp00000.pgp
Description: PGP signature

Reply via email to