[Bf-committers] Finally, a working patch

[joeedh]

Nathan Letwory wrote:

>Actually attaching the patch would help also ;)
>
>/Nathan
>  
>
lol, that's so funny can't believe I forgot that.
-------------- next part --------------
Index: blender/source/blender/blenkernel/BKE_utildefines.h
===================================================================
RCS file: /cvsroot/bf-blender/blender/source/blender/blenkernel/BKE_utildefines.h,v
retrieving revision 1.11
diff -r1.11 BKE_utildefines.h
293a294
> #define TOSEL 4
Index: blender/source/blender/include/editmesh.h
===================================================================
RCS file: /cvsroot/bf-blender/blender/source/blender/include/editmesh.h,v
retrieving revision 1.5
diff -r1.5 editmesh.h
75a76
> extern short extrudeflag_iface(short flag);
Index: blender/source/blender/makesdna/DNA_mesh_types.h
===================================================================
RCS file: /cvsroot/bf-blender/blender/source/blender/makesdna/DNA_mesh_types.h,v
retrieving revision 1.20
diff -r1.20 DNA_mesh_types.h
104c104,105
< 
---
> //individual extrude faces flag
> #define EX_IFACE 1024
165c166
< 
---
> #
Index: blender/source/blender/src/editmesh_lib.c
===================================================================
RCS file: /cvsroot/bf-blender/blender/source/blender/src/editmesh_lib.c,v
retrieving revision 1.8
diff -r1.8 editmesh_lib.c
467a468,565
> static short extrudeflag_iface(short flag)
> {
> 	EditMesh *em = G.editMesh;
> 	EditVert *eve, *nextve;
> 	EditEdge *eed, *nexted;
> 	EditFace *efa, *nextfa;
> 	float nor[3]={0.0, 0.0, 0.0};
> 	short del_old= 0;
> 	
> 	if(G.obedit==0 || get_mesh(G.obedit)==0) return 0;
> 	
> 
> 	/* step 1; init, count faces in edges */
> 	recalc_editnormals();
> 	
> 		
> 	/* step 2: make new faces*/
> 	EditFace *workface;
> 	EditVert *wv1, *wv2, *wv3, *wv4;
> 	EditFace *next = NULL;
> 	efa = em->faces.last;
> 	while (efa) {
> 	    
> 		if(efa->f & SELECT) {
> 	
> 		    wv1 = addvertlist(efa->v1->co);
> 		    VECCOPY(wv1->no, efa->n);
> 		    wv2 = addvertlist(efa->v2->co);
> 		    VECCOPY(wv2->no, efa->n);
> 		    wv3 = addvertlist(efa->v3->co);
> 		    VECCOPY(wv3->no, efa->n);
> 		    if (efa->v4) { 
>                wv4 = addvertlist(efa->v4->co);
> 		       VECCOPY(wv4->no, efa->n);         
> 		    }    
>     		workface = addfacelist(efa->v1, efa->v2, 
>                        wv2, wv1, efa);
>             workface = addfacelist(efa->v2, efa->v3, 
>                        wv3, wv2, efa);
>             if (efa->v4) workface = addfacelist(efa->v3, efa->v4, 
>                        wv4, wv3, efa);
>             else workface = addfacelist(efa->v3, efa->v1, wv1, wv3, efa);
>             
>             if (efa->v4) workface = addfacelist(efa->v4, efa->v1, 
>                        wv1, wv4, efa);
>              
> 			    
>             if (efa->v4)
>     			workface = addfacelist(wv1, wv2, wv3, wv4, efa);
> 		    else
> 		        workface = addfacelist(wv1, wv2, wv3, NULL, efa); 
> 		    
> 		    
> 		    workface->e1 = addedgelist(workface->v1, workface->v2, workface->e1);
> 		    workface->e2 = addedgelist(workface->v2, workface->v3, workface->e2);
> 		    if (!workface->e4) workface->e3 = addedgelist(workface->v3, workface->v1, workface->e3);
> 		    else workface->e3 = addedgelist(workface->v3, workface->v4, workface->e3);
> 		    if (workface->e4) workface->e4 = addedgelist(workface->v4, workface->v1, workface->e4);
> 		    
> 		    workface->f |= SELECT;
>             workface->f |= TOSEL;
>             workface->v1->f1 |= TOSEL;
>             workface->v2->f1 |= TOSEL;
>             workface->v3->f1 |= TOSEL;
>             if (workface->v4) workface->v4->f1 |= TOSEL;
>             
>             //invert the old face since we arn't deleting it
>             efa->n[0] *= -1;
>             efa->n[1] *= -1;
>             efa->n[2] *= -1;    
>             next = efa->prev;
>             BLI_remlink(&em->faces, efa);
> 	        free_editface(efa);        
> 		} else next = NULL;
> 	    if (!next)
> 	      efa = efa->prev;
>         else efa = next;
> 	}
> 	
> 		/* step 7: redo selection */
> 	EM_clear_flag_all(SELECT);
> 
> 	Normalise(nor);	// translation normal grab
> 	
> 	for(eve= em->verts.first; eve; eve= eve->next) {
> 		if(eve->f1 & TOSEL) {
> 			eve->f |= SELECT;
> 			eve->f1 &= ~TOSEL;
> 			
> 		}
> 	 
> 	}
> 
> 	EM_select_flush();
> 	
> 	return 1;
> }
> 
870c968,979
< 	if(G.scene->selectmode & SCE_SELECT_VERTEX)
---
> 	short event;
>     if (G.scene->selectmode & SCE_SELECT_FACE) {
>       event= pupmenu("Extrusion Type %t|Region%x1|Faces%x2");
>       switch (event) {
>           case 1:
>               return extrudeflag_edge(flag);
>               break;
>           case 2:
>               return extrudeflag_iface(flag);
>               break;
>       }    
>     } else if(G.scene->selectmode & SCE_SELECT_VERTEX)
872c981
< 	else 
---
> 	else if (G.scene->selectmode & SCE_SELECT_EDGE)
Index: blender/source/blender/src/editmesh_tools.c
===================================================================
RCS file: /cvsroot/bf-blender/blender/source/blender/src/editmesh_tools.c,v
retrieving revision 1.8
diff -r1.8 editmesh_tools.c
534c534,535
< 
---
>     short event;
>     
551a553,574
> 
> /*void extrude_ifaces_mesh(void)
> {
> 	short a;
> 
> 	TEST_EDITMESH
> 
> 	if(okee("Extrude")==0) return;
> 	
> 	a= extrudeflag(SELECT);
> 
> 	if(a==0) {
> 		error("No valid selection");
> 	}
> 	else {
> 		EM_fgon_flags();
> 		countall();  /* for G.totvert in calc_meshverts() 
> 		calc_meshverts();
> 		transform('n');
> 	}
> 
> }*/
Index: blender/source/blender/src/editobject.c
===================================================================
RCS file: /cvsroot/bf-blender/blender/source/blender/src/editobject.c,v
retrieving revision 1.117
diff -r1.117 editobject.c
5336,5337c5336,5339
< 								
< 								dot= tv->nor[0]*dvecp[0] + tv->nor[1]*dvecp[1] + tv->nor[2]*dvecp[2];
---
> 								//this messes up individual face extrude. . .
> 								//dot= tv->nor[0]*dvecp[0] + tv->nor[1]*dvecp[1] + tv->nor[2]*dvecp[2];
> 								//so average it. . .
> 								dot = (dvecp[0] + dvecp[1] + dvecp[2]) / 3;
8093c8095
< }
\ No newline at end of file
---
> }