>>>>> "Georg" == Georg Baum <[EMAIL PROTECTED]> writes:

Georg> Jean-Marc Lasgouttes wrote:
>> Georg, is this patch correct?

Georg> Oh, I missed this one. Yes, it is correct, and while you are at
Georg> it could you please fix another buglet of mine and move
Georg> MenuItem::UpdateFormats up in the same case as
Georg> MenuItem::ViewFormats and MenuItem::ExportFormats are?
Georg> Otherwise the Update menu entry will contain too many formats.

You mean like that?

JMarc

Index: src/MenuBackend.C
===================================================================
--- src/MenuBackend.C	(revision 13965)
+++ src/MenuBackend.C	(working copy)
@@ -515,10 +515,12 @@ void expandFormats(MenuItem::Kind kind, 
 			break;
 		case MenuItem::ViewFormats:
 		case MenuItem::ExportFormats:
+		case MenuItem::UpdateFormats:
 			if (!(*fit)->documentFormat())
 				continue;
 			break;
-		case MenuItem::UpdateFormats:
+		default:
+			BOOST_ASSERT(false);
 			break;
 		}
 		if (!(*fit)->shortcut().empty())

Reply via email to