compiler warning in MenuBackend

2006-05-30 Thread Jean-Marc Lasgouttes
Georg, is this patch correct? JMarc Index: src/MenuBackend.C === --- src/MenuBackend.C (revision 13965) +++ src/MenuBackend.C (working copy) @@ -520,6 +520,9 @@ void expandFormats(MenuItem::Kind kind, break; case

Re: compiler warning in MenuBackend

2006-05-30 Thread Georg Baum
Jean-Marc Lasgouttes wrote: Georg, is this patch correct? Oh, I missed this one. Yes, it is correct, and while you are at it could you please fix another buglet of mine and move MenuItem::UpdateFormats up in the same case as MenuItem::ViewFormats and MenuItem::ExportFormats are? Otherwise the

Re: compiler warning in MenuBackend

2006-05-30 Thread Jean-Marc Lasgouttes
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

Re: compiler warning in MenuBackend

2006-05-30 Thread Georg Baum
Jean-Marc Lasgouttes wrote: You mean like that? Perfect! Georg

Re: compiler warning in MenuBackend

2006-05-30 Thread Jean-Marc Lasgouttes
Georg == Georg Baum [EMAIL PROTECTED] writes: Georg Jean-Marc Lasgouttes wrote: You mean like that? Georg Perfect! Applied. JMarc

compiler warning in MenuBackend

2006-05-30 Thread Jean-Marc Lasgouttes
Georg, is this patch correct? JMarc Index: src/MenuBackend.C === --- src/MenuBackend.C (revision 13965) +++ src/MenuBackend.C (working copy) @@ -520,6 +520,9 @@ void expandFormats(MenuItem::Kind kind, break; case

Re: compiler warning in MenuBackend

2006-05-30 Thread Georg Baum
Jean-Marc Lasgouttes wrote: > Georg, is this patch correct? Oh, I missed this one. Yes, it is correct, and while you are at it could you please fix another buglet of mine and move MenuItem::UpdateFormats up in the same case as MenuItem::ViewFormats and MenuItem::ExportFormats are? Otherwise the

Re: compiler warning in MenuBackend

2006-05-30 Thread Jean-Marc Lasgouttes
> "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

Re: compiler warning in MenuBackend

2006-05-30 Thread Georg Baum
Jean-Marc Lasgouttes wrote: > You mean like that? Perfect! Georg

Re: compiler warning in MenuBackend

2006-05-30 Thread Jean-Marc Lasgouttes
> "Georg" == Georg Baum <[EMAIL PROTECTED]> writes: Georg> Jean-Marc Lasgouttes wrote: >> You mean like that? Georg> Perfect! Applied. JMarc