Re: PATCH: menu drawing fixes

2005-11-26 Thread Dominik Vogt
On Fri, Nov 25, 2005 at 04:05:31PM -0500, Dan Espen wrote:
 Dan Espen [EMAIL PROTECTED] writes:
  Dominik Vogt [EMAIL PROTECTED] writes:
   The attached patch fixes various menu drawing problems. 
   Would someone be so kind and commit it to CVS, please?
  
  Did you post the whole patch?
  menu_expose in menu.h still returns void.
  I got:
  
  ../../fvwm2_5_15/fvwm/menus.c:6814: error: conflicting types for 
  'menu_expose
  '
  ../../fvwm2_5_15/fvwm/menus.h:355: error: previous declaration of 
  'menu_expos
  e' was here
 
 More errors after fixing that:
 
 ../../fvwm2_5_15/fvwm/menus.c: In function 'make_menu':
 ../../fvwm2_5_15/fvwm/menus.c:2675: warning: implicit declaration of function 
 'MR_USED_MINI_ICONS'
 ../../fvwm2_5_15/fvwm/menus.c:2675: error: invalid lvalue in assignment
 ../../fvwm2_5_15/fvwm/menus.c: In function 'get_menu_paint_item_parameters':
 ../../fvwm2_5_15/fvwm/menus.c:3355: error: 'struct anonymous' has no member 
 named 'used_mini_icons'

Sorry, I forgot to include the header files menus.h and menuitem.h
in the patch.  I've committed them now.

Ciao

Dominik ^_^  ^_^

 --
Dominik Vogt, [EMAIL PROTECTED]


signature.asc
Description: Digital signature


PATCH: menu drawing fixes

2005-11-25 Thread Dominik Vogt
The attached patch fixes various menu drawing problems. 
Would someone be so kind and commit it to CVS, please?

-- 
Ciao

Dominik ^_^  ^_^

Telefonieren Sie schon oder sparen Sie noch?
NEU: GMX Phone_Flat http://www.gmx.net/de/go/telefonie

menudrawing.patch
Description: Binary data


Re: PATCH: menu drawing fixes

2005-11-25 Thread Dan Espen
Dominik Vogt [EMAIL PROTECTED] writes:
 The attached patch fixes various menu drawing problems. 
 Would someone be so kind and commit it to CVS, please?

Did you post the whole patch?
menu_expose in menu.h still returns void.
I got:

../../fvwm2_5_15/fvwm/menus.c:6814: error: conflicting types for 'menu_expose'
../../fvwm2_5_15/fvwm/menus.h:355: error: previous declaration of 'menu_expose' 
was here

-- 
Dan Espen   E-mail: [EMAIL PROTECTED]