tags 357557 + patch
thanks

* Martin Michlmayr <[EMAIL PROTECTED]> [2006-03-18 03:16]:
> I don't know how to fix this one.

Actually, I looked at this some more.  It's 'num' that causes the
problem rather than the other variable (which is what I thought
originally).  Since 'num' is not actually used, how about simply
removing it?  If you do think it's necessary, it has to be declared as
an int explicitly... assigning 0 to something of type
activemenulist::iterator is not going to work.


--- ./src/vscreen/vs_menubar.cc~        2006-03-18 04:43:18.000000000 +0000
+++ ./src/vscreen/vs_menubar.cc 2006-03-18 04:46:21.000000000 +0000
@@ -309,9 +309,9 @@
     }
 
   // Expand the width as needed to account for active menus.
-  for(activemenulist::iterator i=active_menus.begin(), num=0;
+  for(activemenulist::iterator i=active_menus.begin();
       i!=active_menus.end();
-      i++, num++)
+      i++)
     {
       int menux=0;
 
@@ -345,9 +345,9 @@
 
   int h=always_visible?1:0;
 
-  for(activemenulist::iterator i=active_menus.begin(), num=0;
+  for(activemenulist::iterator i=active_menus.begin();
       i!=active_menus.end();
-      i++, num++)
+      i++)
     h=max(h, 1+(*i)->height_request(w));
 
   if(subwidget.valid())


aptitude builds with 4.1 with this and my last patch applied.
-- 
Martin Michlmayr
http://www.cyrius.com/


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to