[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [23735] trunk/blender/source/blender: wm menu freeing was using freed memory, bone-parent names were being set to "" on menu draw WHY??? - r23247, you know who you are ; )

[Campbell Barton]

Revision: 23735
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=23735
Author:   campbellbarton
Date:     2009-10-09 14:34:37 +0200 (Fri, 09 Oct 2009)

Log Message:
-----------
wm menu freeing was using freed memory, bone-parent names were being set to "" on menu draw WHY??? - r23247, you know who you are ;)

Modified Paths:
--------------
    trunk/blender/source/blender/editors/space_view3d/view3d_buttons.c
    trunk/blender/source/blender/windowmanager/intern/wm.c

Modified: trunk/blender/source/blender/editors/space_view3d/view3d_buttons.c
===================================================================
--- trunk/blender/source/blender/editors/space_view3d/view3d_buttons.c	2009-10-09 12:18:32 UTC (rev 23734)
+++ trunk/blender/source/blender/editors/space_view3d/view3d_buttons.c	2009-10-09 12:34:37 UTC (rev 23735)
@@ -1135,7 +1135,6 @@
 	}
 	else {
 		if((ob->mode & OB_MODE_PARTICLE_EDIT)==0) {
-			strcpy(ob->parsubstr, "");
 			uiBlockEndAlign(block);
 		}
 	}

Modified: trunk/blender/source/blender/windowmanager/intern/wm.c
===================================================================
--- trunk/blender/source/blender/windowmanager/intern/wm.c	2009-10-09 12:18:32 UTC (rev 23734)
+++ trunk/blender/source/blender/windowmanager/intern/wm.c	2009-10-09 12:34:37 UTC (rev 23735)
@@ -162,13 +162,17 @@
 
 void WM_menutype_free(void)
 {
-	MenuType* mt;
+	MenuType* mt= menutypes.first, *mt_next;
 
-	for(mt= menutypes.first; mt; mt= mt->next) {
-		if(mt->ext.free) {
+	while(mt) {
+		mt_next= mt->next;
+
+		if(mt->ext.free)
 			mt->ext.free(mt->ext.data);
-		}
+
 		WM_menutype_freelink(mt);
+
+		mt= mt_next;
 	}
 }