style cleanup
[blender.git] / source / blender / bmesh / intern / bmesh_core.c
1 /*
2  * ***** BEGIN GPL LICENSE BLOCK *****
3  *
4  * This program is free software; you can redistribute it and/or
5  * modify it under the terms of the GNU General Public License
6  * as published by the Free Software Foundation; either version 2
7  * of the License, or (at your option) any later version.
8  *
9  * This program is distributed in the hope that it will be useful,
10  * but WITHOUT ANY WARRANTY; without even the implied warranty of
11  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
12  * GNU General Public License for more details.
13  *
14  * You should have received a copy of the GNU General Public License
15  * along with this program; if not, write to the Free Software Foundation,
16  * Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
17  *
18  * Contributor(s): Joseph Eagar, Geoffrey Bantle, Campbell Barton
19  *
20  * ***** END GPL LICENSE BLOCK *****
21  */
22
23 /** \file blender/bmesh/intern/bmesh_core.c
24  *  \ingroup bmesh
25  *
26  */
27
28 #include "MEM_guardedalloc.h"
29
30 #include "BLI_math_vector.h"
31
32 #include "BKE_DerivedMesh.h"
33
34 #include "BLI_listbase.h"
35 #include "BLI_array.h"
36
37 #include "bmesh.h"
38 #include "intern/bmesh_private.h"
39
40 /* use so valgrinds memcheck alerts us when undefined index is used.
41  * TESTING ONLY! */
42 // #define USE_DEBUG_INDEX_MEMCHECK
43
44 #ifdef USE_DEBUG_INDEX_MEMCHECK
45 #define DEBUG_MEMCHECK_INDEX_INVALIDATE(ele)                                  \
46         {                                                                         \
47                 int undef_idx;                                                        \
48                 BM_elem_index_set(ele, undef_idx); /* set_ok_invalid */               \
49         } (void)0
50
51 #endif
52
53 /**
54  * \brief Main function for creating a new vertex.
55  */
56 BMVert *BM_vert_create(BMesh *bm, const float co[3], const BMVert *example)
57 {
58         BMVert *v = BLI_mempool_calloc(bm->vpool);
59
60 #ifdef USE_DEBUG_INDEX_MEMCHECK
61         DEBUG_MEMCHECK_INDEX_INVALIDATE(v)
62 #else
63         BM_elem_index_set(v, -1); /* set_ok_invalid */
64 #endif
65
66         bm->elem_index_dirty |= BM_VERT; /* may add to middle of the pool */
67
68         bm->totvert++;
69
70         v->head.htype = BM_VERT;
71
72         /* 'v->no' is handled by BM_elem_attrs_copy */
73         if (co) {
74                 copy_v3_v3(v->co, co);
75         }
76
77         /* allocate flag */
78         v->oflags = BLI_mempool_calloc(bm->toolflagpool);
79
80         CustomData_bmesh_set_default(&bm->vdata, &v->head.data);
81         
82         if (example) {
83                 int *keyi;
84
85                 BM_elem_attrs_copy(bm, bm, example, v);
86
87                 /* exception: don't copy the original shapekey index */
88                 keyi = CustomData_bmesh_get(&bm->vdata, v->head.data, CD_SHAPE_KEYINDEX);
89                 if (keyi) {
90                         *keyi = ORIGINDEX_NONE;
91                 }
92         }
93
94         BM_CHECK_ELEMENT(v);
95
96         return v;
97 }
98
99 /**
100  * \brief Main function for creating a new edge.
101  *
102  * \note Duplicate edges are supported by the API however users should _never_ see them.
103  * so unless you need a unique edge or know the edge won't exist, you should call with \a nodouble = TRUE
104  */
105 BMEdge *BM_edge_create(BMesh *bm, BMVert *v1, BMVert *v2, const BMEdge *example, int nodouble)
106 {
107         BMEdge *e;
108         
109         if (nodouble && (e = BM_edge_exists(v1, v2)))
110                 return e;
111         
112         e = BLI_mempool_calloc(bm->epool);
113
114 #ifdef USE_DEBUG_INDEX_MEMCHECK
115         DEBUG_MEMCHECK_INDEX_INVALIDATE(e)
116 #else
117         BM_elem_index_set(e, -1); /* set_ok_invalid */
118 #endif
119
120         bm->elem_index_dirty |= BM_EDGE; /* may add to middle of the pool */
121
122         bm->totedge++;
123
124         e->head.htype = BM_EDGE;
125         
126         /* allocate flag */
127         e->oflags = BLI_mempool_calloc(bm->toolflagpool);
128
129         e->v1 = v1;
130         e->v2 = v2;
131         
132         BM_elem_flag_enable(e, BM_ELEM_SMOOTH);
133         
134         CustomData_bmesh_set_default(&bm->edata, &e->head.data);
135         
136         bmesh_disk_edge_append(e, e->v1);
137         bmesh_disk_edge_append(e, e->v2);
138         
139         if (example)
140                 BM_elem_attrs_copy(bm, bm, example, e);
141         
142         BM_CHECK_ELEMENT(e);
143
144         return e;
145 }
146
147 static BMLoop *bm_loop_create(BMesh *bm, BMVert *v, BMEdge *e, BMFace *f, const BMLoop *example)
148 {
149         BMLoop *l = NULL;
150
151         l = BLI_mempool_calloc(bm->lpool);
152         l->next = l->prev = NULL;
153         l->v = v;
154         l->e = e;
155         l->f = f;
156         l->radial_next = l->radial_prev = NULL;
157         l->head.data = NULL;
158         l->head.htype = BM_LOOP;
159
160         bm->totloop++;
161
162         if (example) {
163                 CustomData_bmesh_copy_data(&bm->ldata, &bm->ldata, example->head.data, &l->head.data);
164         }
165         else {
166                 CustomData_bmesh_set_default(&bm->ldata, &l->head.data);
167         }
168
169         return l;
170 }
171
172 static BMLoop *bm_face_boundary_add(BMesh *bm, BMFace *f, BMVert *startv, BMEdge *starte)
173 {
174 #ifdef USE_BMESH_HOLES
175         BMLoopList *lst = BLI_mempool_calloc(bm->looplistpool);
176 #endif
177         BMLoop *l = bm_loop_create(bm, startv, starte, f, NULL);
178         
179         bmesh_radial_append(starte, l);
180
181 #ifdef USE_BMESH_HOLES
182         lst->first = lst->last = l;
183         BLI_addtail(&f->loops, lst);
184 #else
185         f->l_first = l;
186 #endif
187
188         l->f = f;
189         
190         return l;
191 }
192
193 BMFace *BM_face_copy(BMesh *bm, BMFace *f, const short copyverts, const short copyedges)
194 {
195         BMVert **verts = NULL;
196         BMEdge **edges = NULL;
197         BLI_array_fixedstack_declare(verts, BM_NGON_STACK_SIZE, f->len, __func__);
198         BLI_array_fixedstack_declare(edges, BM_NGON_STACK_SIZE, f->len, __func__);
199         BMLoop *l_iter;
200         BMLoop *l_first;
201         BMLoop *l_copy;
202         BMFace *f_copy;
203         int i;
204
205         l_iter = l_first = BM_FACE_FIRST_LOOP(f);
206         i = 0;
207         do {
208                 if (copyverts) {
209                         verts[i] = BM_vert_create(bm, l_iter->v->co, l_iter->v);
210                 }
211                 else {
212                         verts[i] = l_iter->v;
213                 }
214                 i++;
215         } while ((l_iter = l_iter->next) != l_first);
216
217         l_iter = l_first = BM_FACE_FIRST_LOOP(f);
218         i = 0;
219         do {
220                 if (copyedges) {
221                         BMVert *v1, *v2;
222                         
223                         if (l_iter->e->v1 == verts[i]) {
224                                 v1 = verts[i];
225                                 v2 = verts[(i + 1) % f->len];
226                         }
227                         else {
228                                 v2 = verts[i];
229                                 v1 = verts[(i + 1) % f->len];
230                         }
231                         
232                         edges[i] = BM_edge_create(bm,  v1, v2, l_iter->e, FALSE);
233                 }
234                 else {
235                         edges[i] = l_iter->e;
236                 }
237                 i++;
238         } while ((l_iter = l_iter->next) != l_first);
239         
240         f_copy = BM_face_create(bm, verts, edges, f->len, FALSE);
241         
242         BM_elem_attrs_copy(bm, bm, f, f_copy);
243         
244         l_iter = l_first = BM_FACE_FIRST_LOOP(f);
245         l_copy = BM_FACE_FIRST_LOOP(f_copy);
246         do {
247                 BM_elem_attrs_copy(bm, bm, l_iter, l_copy);
248                 l_copy = l_copy->next;
249         } while ((l_iter = l_iter->next) != l_first);
250
251         BLI_array_fixedstack_free(verts);
252         BLI_array_fixedstack_free(edges);
253
254         return f_copy;
255 }
256
257 /**
258  * only create the face, since this calloc's the length is initialized to 0,
259  * leave adding loops to the caller.
260  */
261 BLI_INLINE BMFace *bm_face_create__internal(BMesh *bm)
262 {
263         BMFace *f;
264
265         f = BLI_mempool_calloc(bm->fpool);
266
267 #ifdef USE_DEBUG_INDEX_MEMCHECK
268         DEBUG_MEMCHECK_INDEX_INVALIDATE(f)
269 #else
270         BM_elem_index_set(f, -1); /* set_ok_invalid */
271 #endif
272
273         bm->elem_index_dirty |= BM_FACE; /* may add to middle of the pool */
274
275         bm->totface++;
276
277         f->head.htype = BM_FACE;
278
279         /* allocate flag */
280         f->oflags = BLI_mempool_calloc(bm->toolflagpool);
281
282         CustomData_bmesh_set_default(&bm->pdata, &f->head.data);
283
284 #ifdef USE_BMESH_HOLES
285         f->totbounds = 0;
286 #endif
287
288         return f;
289 }
290
291 /**
292  * \brief Main face creation function
293  */
294 BMFace *BM_face_create(BMesh *bm, BMVert **verts, BMEdge **edges, const int len, int nodouble)
295 {
296         BMFace *f = NULL;
297         BMLoop *l, *startl, *lastl;
298         int i, overlap;
299         
300         if (len == 0) {
301                 /* just return NULL for no */
302                 return NULL;
303         }
304
305         if (nodouble) {
306                 /* Check if face already exists */
307                 overlap = BM_face_exists(bm, verts, len, &f);
308                 if (overlap) {
309                         return f;
310                 }
311                 else {
312                         BLI_assert(f == NULL);
313                 }
314         }
315
316         f = bm_face_create__internal(bm);
317
318         startl = lastl = bm_face_boundary_add(bm, f, verts[0], edges[0]);
319         
320         startl->v = verts[0];
321         startl->e = edges[0];
322         for (i = 1; i < len; i++) {
323                 l = bm_loop_create(bm, verts[i], edges[i], f, edges[i]->l);
324                 
325                 l->f = f;
326                 bmesh_radial_append(edges[i], l);
327
328                 l->prev = lastl;
329                 lastl->next = l;
330                 lastl = l;
331         }
332         
333         startl->prev = lastl;
334         lastl->next = startl;
335         
336         f->len = len;
337         
338         BM_CHECK_ELEMENT(f);
339
340         return f;
341 }
342
343 /**
344  * Check the element is valid.
345  *
346  * BMESH_TODO, when this raises an error the output is incredible confusing.
347  * need to have some nice way to print/debug what the hecks going on.
348  */
349 int bmesh_elem_check(void *element, const char htype)
350 {
351         BMHeader *head = element;
352         int err = 0;
353
354         if (!element)
355                 return 1;
356
357         if (head->htype != htype)
358                 return 2;
359         
360         switch (htype) {
361                 case BM_VERT:
362                 {
363                         BMVert *v = element;
364                         if (v->e && v->e->head.htype != BM_EDGE) {
365                                 err |= 4;
366                         }
367                         break;
368                 }
369                 case BM_EDGE:
370                 {
371                         BMEdge *e = element;
372                         if (e->l && e->l->head.htype != BM_LOOP)
373                                 err |= 8;
374                         if (e->l && e->l->f->head.htype != BM_FACE)
375                                 err |= 16;
376                         if (e->v1_disk_link.prev == NULL ||
377                             e->v2_disk_link.prev == NULL ||
378                             e->v1_disk_link.next == NULL ||
379                             e->v2_disk_link.next == NULL)
380                         {
381                                 err |= 32;
382                         }
383                         if (e->l && (e->l->radial_next == NULL || e->l->radial_prev == NULL))
384                                 err |= 64;
385                         if (e->l && e->l->f->len <= 0)
386                                 err |= 128;
387                         break;
388                 }
389                 case BM_LOOP:
390                 {
391                         BMLoop *l = element, *l2;
392                         int i;
393
394                         if (l->f->head.htype != BM_FACE)
395                                 err |= 256;
396                         if (l->e->head.htype != BM_EDGE)
397                                 err |= 512;
398                         if (l->v->head.htype != BM_VERT)
399                                 err |= 1024;
400                         if (!BM_vert_in_edge(l->e, l->v)) {
401                                 fprintf(stderr, "%s: fatal bmesh error (vert not in edge)! (bmesh internal error)\n", __func__);
402                                 err |= 2048;
403                         }
404
405                         if (l->radial_next == NULL || l->radial_prev == NULL)
406                                 err |= (1 << 12);
407                         if (l->f->len <= 0)
408                                 err |= (1 << 13);
409
410                         /* validate boundary loop -- invalid for hole loops, of course,
411                          * but we won't be allowing those for a while yet */
412                         l2 = l;
413                         i = 0;
414                         do {
415                                 if (i >= BM_NGON_MAX) {
416                                         break;
417                                 }
418
419                                 i++;
420                         } while ((l2 = l2->next) != l);
421
422                         if (i != l->f->len || l2 != l)
423                                 err |= (1 << 14);
424
425                         if (!bmesh_radial_validate(bmesh_radial_length(l), l))
426                                 err |= (1 << 15);
427
428                         break;
429                 }
430                 case BM_FACE:
431                 {
432                         BMFace *f = element;
433                         BMLoop *l_iter;
434                         BMLoop *l_first;
435                         int len = 0;
436
437 #ifdef USE_BMESH_HOLES
438                         if (!f->loops.first)
439 #else
440                         if (!f->l_first)
441 #endif
442                         {
443                                 err |= (1 << 16);
444                         }
445                         l_iter = l_first = BM_FACE_FIRST_LOOP(f);
446                         do {
447                                 if (l_iter->f != f) {
448                                         fprintf(stderr, "%s: loop inside one face points to another! (bmesh internal error)\n", __func__);
449                                         err |= (1 << 17);
450                                 }
451
452                                 if (!l_iter->e)
453                                         err |= (1 << 18);
454                                 if (!l_iter->v)
455                                         err |= (1 << 19);
456                                 if (!BM_vert_in_edge(l_iter->e, l_iter->v) || !BM_vert_in_edge(l_iter->e, l_iter->next->v)) {
457                                         err |= (1 << 20);
458                                 }
459
460                                 if (!bmesh_radial_validate(bmesh_radial_length(l_iter), l_iter))
461                                         err |= (1 << 21);
462
463                                 if (!bmesh_disk_count(l_iter->v) || !bmesh_disk_count(l_iter->next->v))
464                                         err |= (1 << 22);
465
466                                 len++;
467                         } while ((l_iter = l_iter->next) != l_first);
468
469                         if (len != f->len)
470                                 err |= (1 << 23);
471                 }
472         }
473
474         BMESH_ASSERT(err == 0);
475
476         return err;
477 }
478
479 /**
480  * low level function, only frees the vert,
481  * doesn't change or adjust surrounding geometry
482  */
483 static void bm_kill_only_vert(BMesh *bm, BMVert *v)
484 {
485         bm->totvert--;
486         bm->elem_index_dirty |= BM_VERT;
487
488         BM_select_history_remove(bm, v);
489
490         if (v->head.data)
491                 CustomData_bmesh_free_block(&bm->vdata, &v->head.data);
492
493         BLI_mempool_free(bm->toolflagpool, v->oflags);
494         BLI_mempool_free(bm->vpool, v);
495 }
496
497 /**
498  * low level function, only frees the edge,
499  * doesn't change or adjust surrounding geometry
500  */
501 static void bm_kill_only_edge(BMesh *bm, BMEdge *e)
502 {
503         bm->totedge--;
504         bm->elem_index_dirty |= BM_EDGE;
505
506         BM_select_history_remove(bm, (BMElem *)e);
507
508         if (e->head.data)
509                 CustomData_bmesh_free_block(&bm->edata, &e->head.data);
510
511         BLI_mempool_free(bm->toolflagpool, e->oflags);
512         BLI_mempool_free(bm->epool, e);
513 }
514
515 /**
516  * low level function, only frees the face,
517  * doesn't change or adjust surrounding geometry
518  */
519 static void bm_kill_only_face(BMesh *bm, BMFace *f)
520 {
521         if (bm->act_face == f)
522                 bm->act_face = NULL;
523
524         bm->totface--;
525         bm->elem_index_dirty |= BM_FACE;
526
527         BM_select_history_remove(bm, (BMElem *)f);
528
529         if (f->head.data)
530                 CustomData_bmesh_free_block(&bm->pdata, &f->head.data);
531
532         BLI_mempool_free(bm->toolflagpool, f->oflags);
533         BLI_mempool_free(bm->fpool, f);
534 }
535
536 /**
537  * low level function, only frees the loop,
538  * doesn't change or adjust surrounding geometry
539  */
540 static void bm_kill_only_loop(BMesh *bm, BMLoop *l)
541 {
542         bm->totloop--;
543         if (l->head.data)
544                 CustomData_bmesh_free_block(&bm->ldata, &l->head.data);
545
546         BLI_mempool_free(bm->lpool, l);
547 }
548
549 /**
550  * kills all edges associated with \a f, along with any other faces containing
551  * those edges
552  */
553 void BM_face_edges_kill(BMesh *bm, BMFace *f)
554 {
555         BMEdge **edges = NULL;
556         BLI_array_staticdeclare(edges, BM_NGON_STACK_SIZE);
557         BMLoop *l_iter;
558         BMLoop *l_first;
559         int i;
560         
561         l_iter = l_first = BM_FACE_FIRST_LOOP(f);
562         do {
563                 BLI_array_append(edges, l_iter->e);
564         } while ((l_iter = l_iter->next) != l_first);
565         
566         for (i = 0; i < BLI_array_count(edges); i++) {
567                 BM_edge_kill(bm, edges[i]);
568         }
569         
570         BLI_array_free(edges);
571 }
572
573 /**
574  * kills all verts associated with \a f, along with any other faces containing
575  * those vertices
576  */
577 void BM_face_verts_kill(BMesh *bm, BMFace *f)
578 {
579         BMVert **verts = NULL;
580         BLI_array_staticdeclare(verts, BM_NGON_STACK_SIZE);
581         BMLoop *l_iter;
582         BMLoop *l_first;
583         int i;
584         
585         l_iter = l_first = BM_FACE_FIRST_LOOP(f);
586         do {
587                 BLI_array_append(verts, l_iter->v);
588         } while ((l_iter = l_iter->next) != l_first);
589         
590         for (i = 0; i < BLI_array_count(verts); i++) {
591                 BM_vert_kill(bm, verts[i]);
592         }
593         
594         BLI_array_free(verts);
595 }
596
597 /**
598  * Kills \a f and its loops.
599  */
600 void BM_face_kill(BMesh *bm, BMFace *f)
601 {
602 #ifdef USE_BMESH_HOLES
603         BMLoopList *ls, *ls_next;
604 #endif
605
606         BM_CHECK_ELEMENT(f);
607
608 #ifdef USE_BMESH_HOLES
609         for (ls = f->loops.first; ls; ls = ls_next)
610 #else
611         if (f->l_first)
612 #endif
613         {
614                 BMLoop *l_iter, *l_next, *l_first;
615
616 #ifdef USE_BMESH_HOLES
617                 ls_next = ls->next;
618                 l_iter = l_first = ls->first;
619 #else
620                 l_iter = l_first = f->l_first;
621 #endif
622
623                 do {
624                         l_next = l_iter->next;
625
626                         bmesh_radial_loop_remove(l_iter, l_iter->e);
627                         bm_kill_only_loop(bm, l_iter);
628
629                 } while ((l_iter = l_next) != l_first);
630
631 #ifdef USE_BMESH_HOLES
632                 BLI_mempool_free(bm->looplistpool, ls);
633 #endif
634         }
635
636         bm_kill_only_face(bm, f);
637 }
638 /**
639  * kills \a e and all faces that use it.
640  */
641 void BM_edge_kill(BMesh *bm, BMEdge *e)
642 {
643
644         bmesh_disk_edge_remove(e, e->v1);
645         bmesh_disk_edge_remove(e, e->v2);
646
647         if (e->l) {
648                 BMLoop *l = e->l, *lnext, *startl = e->l;
649
650                 do {
651                         lnext = l->radial_next;
652                         if (lnext->f == l->f) {
653                                 BM_face_kill(bm, l->f);
654                                 break;
655                         }
656                         
657                         BM_face_kill(bm, l->f);
658
659                         if (l == lnext)
660                                 break;
661                         l = lnext;
662                 } while (l != startl);
663         }
664         
665         bm_kill_only_edge(bm, e);
666 }
667
668 /**
669  * kills \a v and all edges that use it.
670  */
671 void BM_vert_kill(BMesh *bm, BMVert *v)
672 {
673         if (v->e) {
674                 BMEdge *e, *nexte;
675                 
676                 e = v->e;
677                 while (v->e) {
678                         nexte = bmesh_disk_edge_next(e, v);
679                         BM_edge_kill(bm, e);
680                         e = nexte;
681                 }
682         }
683
684         bm_kill_only_vert(bm, v);
685 }
686
687 /********** private disk and radial cycle functions ********** */
688
689 /**
690  * return the length of the face, should always equal \a l->f->len
691  */
692 static int UNUSED_FUNCTION(bm_loop_length)(BMLoop *l)
693 {
694         BMLoop *l_first = l;
695         int i = 0;
696
697         do {
698                 i++;
699         } while ((l = l->next) != l_first);
700
701         return i;
702 }
703
704 /**
705  * \brief Loop Reverse
706  *
707  * Changes the winding order of a face from CW to CCW or vice versa.
708  * This euler is a bit peculiar in comparison to others as it is its
709  * own inverse.
710  *
711  * BMESH_TODO: reinsert validation code.
712  *
713  * \return Success
714  */
715 static int bm_loop_reverse_loop(BMesh *bm, BMFace *f
716 #ifdef USE_BMESH_HOLES
717                                 , BMLoopList *lst
718 #endif
719                                 )
720 {
721
722 #ifdef USE_BMESH_HOLES
723         BMLoop *l_first = lst->first;
724 #else
725         BMLoop *l_first = f->l_first;
726 #endif
727
728         const int len = f->len;
729         const int do_disps = CustomData_has_layer(&bm->ldata, CD_MDISPS);
730         BMLoop *l_iter, *oldprev, *oldnext;
731         BMEdge **edar = NULL;
732         BLI_array_fixedstack_declare(edar, BM_NGON_STACK_SIZE, len, __func__);
733         int i, j, edok;
734
735         for (i = 0, l_iter = l_first; i < len; i++, l_iter = l_iter->next) {
736                 bmesh_radial_loop_remove(l_iter, (edar[i] = l_iter->e));
737         }
738
739         /* actually reverse the loop */
740         for (i = 0, l_iter = l_first; i < len; i++) {
741                 oldnext = l_iter->next;
742                 oldprev = l_iter->prev;
743                 l_iter->next = oldprev;
744                 l_iter->prev = oldnext;
745                 l_iter = oldnext;
746                 
747                 if (do_disps) {
748                         float (*co)[3];
749                         int x, y, sides;
750                         MDisps *md;
751                         
752                         md = CustomData_bmesh_get(&bm->ldata, l_iter->head.data, CD_MDISPS);
753                         if (!md->totdisp || !md->disps)
754                                 continue;
755
756                         sides = (int)sqrt(md->totdisp);
757                         co = md->disps;
758                         
759                         for (x = 0; x < sides; x++) {
760                                 for (y = 0; y < x; y++) {
761                                         swap_v3_v3(co[y * sides + x], co[sides * x + y]);
762                                 }
763                         }
764                 }
765         }
766
767         if (len == 2) { /* two edged face */
768                 /* do some verification here! */
769                 l_first->e = edar[1];
770                 l_first->next->e = edar[0];
771         }
772         else {
773                 for (i = 0, l_iter = l_first; i < len; i++, l_iter = l_iter->next) {
774                         edok = 0;
775                         for (j = 0; j < len; j++) {
776                                 edok = bmesh_verts_in_edge(l_iter->v, l_iter->next->v, edar[j]);
777                                 if (edok) {
778                                         l_iter->e = edar[j];
779                                         break;
780                                 }
781                         }
782                 }
783         }
784         /* rebuild radia */
785         for (i = 0, l_iter = l_first; i < len; i++, l_iter = l_iter->next)
786                 bmesh_radial_append(l_iter->e, l_iter);
787
788         /* validate radia */
789         for (i = 0, l_iter = l_first; i < len; i++, l_iter = l_iter->next) {
790                 BM_CHECK_ELEMENT(l_iter);
791                 BM_CHECK_ELEMENT(l_iter->e);
792                 BM_CHECK_ELEMENT(l_iter->v);
793                 BM_CHECK_ELEMENT(l_iter->f);
794         }
795
796         BLI_array_fixedstack_free(edar);
797
798         BM_CHECK_ELEMENT(f);
799
800         return 1;
801 }
802
803 /**
804  * \brief Flip the faces direction
805  */
806 int bmesh_loop_reverse(BMesh *bm, BMFace *f)
807 {
808 #ifdef USE_BMESH_HOLES
809         return bmesh_loop_reverse_loop(bm, f, f->loops.first);
810 #else
811         return bm_loop_reverse_loop(bm, f);
812 #endif
813 }
814
815 static void bm_elements_systag_enable(void *veles, int tot, int flag)
816 {
817         BMHeader **eles = veles;
818         int i;
819
820         for (i = 0; i < tot; i++) {
821                 BM_ELEM_API_FLAG_ENABLE((BMElemF *)eles[i], flag);
822         }
823 }
824
825 static void bm_elements_systag_disable(void *veles, int tot, int flag)
826 {
827         BMHeader **eles = veles;
828         int i;
829
830         for (i = 0; i < tot; i++) {
831                 BM_ELEM_API_FLAG_DISABLE((BMElemF *)eles[i], flag);
832         }
833 }
834
835 #define FACE_MARK  (1 << 10)
836
837 static int count_flagged_radial(BMesh *bm, BMLoop *l, int flag)
838 {
839         BMLoop *l2 = l;
840         int i = 0, c = 0;
841
842         do {
843                 if (UNLIKELY(!l2)) {
844                         BMESH_ASSERT(0);
845                         goto error;
846                 }
847                 
848                 i += BM_ELEM_API_FLAG_TEST(l2->f, flag) ? 1 : 0;
849                 l2 = l2->radial_next;
850                 if (UNLIKELY(c >= BM_LOOP_RADIAL_MAX)) {
851                         BMESH_ASSERT(0);
852                         goto error;
853                 }
854                 c++;
855         } while (l2 != l);
856
857         return i;
858
859 error:
860         BMO_error_raise(bm, bm->currentop, BMERR_MESH_ERROR, NULL);
861         return 0;
862 }
863
864 static int UNUSED_FUNCTION(count_flagged_disk)(BMVert *v, int flag)
865 {
866         BMEdge *e = v->e;
867         int i = 0;
868
869         if (!e)
870                 return 0;
871
872         do {
873                 i += BM_ELEM_API_FLAG_TEST(e, flag) ? 1 : 0;
874         } while ((e = bmesh_disk_edge_next(e, v)) != v->e);
875
876         return i;
877 }
878
879 static int disk_is_flagged(BMVert *v, int flag)
880 {
881         BMEdge *e = v->e;
882
883         if (!e)
884                 return FALSE;
885
886         do {
887                 BMLoop *l = e->l;
888
889                 if (!l) {
890                         return FALSE;
891                 }
892                 
893                 if (bmesh_radial_length(l) == 1)
894                         return FALSE;
895                 
896                 do {
897                         if (!BM_ELEM_API_FLAG_TEST(l->f, flag))
898                                 return FALSE;
899
900                         l = l->radial_next;
901                 } while (l != e->l);
902
903                 e = bmesh_disk_edge_next(e, v);
904         } while (e != v->e);
905
906         return TRUE;
907 }
908
909 /* Mid-level Topology Manipulation Functions */
910
911 /**
912  * \brief Join Connected Faces
913  *
914  * Joins a collected group of faces into one. Only restriction on
915  * the input data is that the faces must be connected to each other.
916  *
917  * \return The newly created combine BMFace.
918  *
919  * \note If a pair of faces share multiple edges,
920  * the pair of faces will be joined at every edge.
921  *
922  * \note this is a generic, flexible join faces function,
923  * almost everything uses this, including #BM_faces_join_pair
924  */
925 BMFace *BM_faces_join(BMesh *bm, BMFace **faces, int totface, const short do_del)
926 {
927         BMFace *f, *newf;
928 #ifdef USE_BMESH_HOLES
929         BMLoopList *lst;
930         ListBase holes = {NULL, NULL};
931 #endif
932         BMLoop *l_iter;
933         BMLoop *l_first;
934         BMEdge **edges = NULL;
935         BMEdge **deledges = NULL;
936         BMVert **delverts = NULL;
937         BLI_array_staticdeclare(edges,    BM_NGON_STACK_SIZE);
938         BLI_array_staticdeclare(deledges, BM_NGON_STACK_SIZE);
939         BLI_array_staticdeclare(delverts, BM_NGON_STACK_SIZE);
940         BMVert *v1 = NULL, *v2 = NULL;
941         const char *err = NULL;
942         int i, tote = 0;
943
944         if (UNLIKELY(!totface)) {
945                 BMESH_ASSERT(0);
946                 return NULL;
947         }
948
949         if (totface == 1)
950                 return faces[0];
951
952         bm_elements_systag_enable(faces, totface, _FLAG_JF);
953
954         for (i = 0; i < totface; i++) {
955                 f = faces[i];
956                 l_iter = l_first = BM_FACE_FIRST_LOOP(f);
957                 do {
958                         int rlen = count_flagged_radial(bm, l_iter, _FLAG_JF);
959
960                         if (rlen > 2) {
961                                 err = "Input faces do not form a contiguous manifold region";
962                                 goto error;
963                         }
964                         else if (rlen == 1) {
965                                 BLI_array_append(edges, l_iter->e);
966
967                                 if (!v1) {
968                                         v1 = l_iter->v;
969                                         v2 = BM_edge_other_vert(l_iter->e, l_iter->v);
970                                 }
971                                 tote++;
972                         }
973                         else if (rlen == 2) {
974                                 int d1, d2;
975
976                                 d1 = disk_is_flagged(l_iter->e->v1, _FLAG_JF);
977                                 d2 = disk_is_flagged(l_iter->e->v2, _FLAG_JF);
978
979                                 if (!d1 && !d2 && !BM_ELEM_API_FLAG_TEST(l_iter->e, _FLAG_JF)) {
980                                         /* don't remove an edge it makes up the side of another face
981                                          * else this will remove the face as well - campbell */
982                                         if (BM_edge_face_count(l_iter->e) <= 2) {
983                                                 if (do_del) {
984                                                         BLI_array_append(deledges, l_iter->e);
985                                                 }
986                                                 BM_ELEM_API_FLAG_ENABLE(l_iter->e, _FLAG_JF);
987                                         }
988                                 }
989                                 else {
990                                         if (d1 && !BM_ELEM_API_FLAG_TEST(l_iter->e->v1, _FLAG_JF)) {
991                                                 if (do_del) {
992                                                         BLI_array_append(delverts, l_iter->e->v1);
993                                                 }
994                                                 BM_ELEM_API_FLAG_ENABLE(l_iter->e->v1, _FLAG_JF);
995                                         }
996
997                                         if (d2 && !BM_ELEM_API_FLAG_TEST(l_iter->e->v2, _FLAG_JF)) {
998                                                 if (do_del) {
999                                                         BLI_array_append(delverts, l_iter->e->v2);
1000                                                 }
1001                                                 BM_ELEM_API_FLAG_ENABLE(l_iter->e->v2, _FLAG_JF);
1002                                         }
1003                                 }
1004                         }
1005                 } while ((l_iter = l_iter->next) != l_first);
1006
1007 #ifdef USE_BMESH_HOLES
1008                 for (lst = f->loops.first; lst; lst = lst->next) {
1009                         if (lst == f->loops.first) {
1010                                 continue;
1011                         }
1012
1013                         BLI_remlink(&f->loops, lst);
1014                         BLI_addtail(&holes, lst);
1015                 }
1016 #endif
1017
1018         }
1019
1020         /* create region face */
1021         newf = BM_face_create_ngon(bm, v1, v2, edges, tote, FALSE);
1022         if (!newf || BMO_error_occurred(bm)) {
1023                 if (!BMO_error_occurred(bm))
1024                         err = "Invalid boundary region to join faces";
1025                 goto error;
1026         }
1027
1028         /* copy over loop data */
1029         l_iter = l_first = BM_FACE_FIRST_LOOP(newf);
1030         do {
1031                 BMLoop *l2 = l_iter->radial_next;
1032
1033                 do {
1034                         if (BM_ELEM_API_FLAG_TEST(l2->f, _FLAG_JF))
1035                                 break;
1036                         l2 = l2->radial_next;
1037                 } while (l2 != l_iter);
1038
1039                 if (l2 != l_iter) {
1040                         /* I think this is correct */
1041                         if (l2->v != l_iter->v) {
1042                                 l2 = l2->next;
1043                         }
1044
1045                         BM_elem_attrs_copy(bm, bm, l2, l_iter);
1046                 }
1047         } while ((l_iter = l_iter->next) != l_first);
1048         
1049         BM_elem_attrs_copy(bm, bm, faces[0], newf);
1050
1051 #ifdef USE_BMESH_HOLES
1052         /* add hole */
1053         BLI_movelisttolist(&newf->loops, &holes);
1054 #endif
1055
1056         /* update loop face pointer */
1057 #ifdef USE_BMESH_HOLES
1058         for (lst = newf->loops.first; lst; lst = lst->next)
1059 #endif
1060         {
1061 #ifdef USE_BMESH_HOLES
1062                 l_iter = l_first = lst->first;
1063 #else
1064                 l_iter = l_first = BM_FACE_FIRST_LOOP(newf);
1065 #endif
1066                 do {
1067                         l_iter->f = newf;
1068                 } while ((l_iter = l_iter->next) != l_first);
1069         }
1070
1071         bm_elements_systag_disable(faces, totface, _FLAG_JF);
1072         BM_ELEM_API_FLAG_DISABLE(newf, _FLAG_JF);
1073
1074         /* handle multi-res data */
1075         if (CustomData_has_layer(&bm->ldata, CD_MDISPS)) {
1076                 l_iter = l_first = BM_FACE_FIRST_LOOP(newf);
1077                 do {
1078                         for (i = 0; i < totface; i++) {
1079                                 BM_loop_interp_multires(bm, l_iter, faces[i]);
1080                         }
1081                 } while ((l_iter = l_iter->next) != l_first);
1082         }
1083
1084         /* delete old geometry */
1085         if (do_del) {
1086                 for (i = 0; i < BLI_array_count(deledges); i++) {
1087                         BM_edge_kill(bm, deledges[i]);
1088                 }
1089
1090                 for (i = 0; i < BLI_array_count(delverts); i++) {
1091                         BM_vert_kill(bm, delverts[i]);
1092                 }
1093         }
1094         else {
1095                 /* otherwise we get both old and new faces */
1096                 for (i = 0; i < totface; i++) {
1097                         BM_face_kill(bm, faces[i]);
1098                 }
1099         }
1100         
1101         BLI_array_free(edges);
1102         BLI_array_free(deledges);
1103         BLI_array_free(delverts);
1104
1105         BM_CHECK_ELEMENT(newf);
1106         return newf;
1107
1108 error:
1109         bm_elements_systag_disable(faces, totface, _FLAG_JF);
1110         BLI_array_free(edges);
1111         BLI_array_free(deledges);
1112         BLI_array_free(delverts);
1113
1114         if (err) {
1115                 BMO_error_raise(bm, bm->currentop, BMERR_DISSOLVEFACES_FAILED, err);
1116         }
1117         return NULL;
1118 }
1119
1120 static BMFace *bm_face_create__sfme(BMesh *bm, BMFace *UNUSED(example))
1121 {
1122         BMFace *f;
1123 #ifdef USE_BMESH_HOLES
1124         BMLoopList *lst;
1125 #endif
1126
1127         f = bm_face_create__internal(bm);
1128
1129 #ifdef USE_BMESH_HOLES
1130         lst = BLI_mempool_calloc(bm->looplistpool);
1131         BLI_addtail(&f->loops, lst);
1132 #endif
1133
1134 #ifdef USE_BMESH_HOLES
1135         f->totbounds = 1;
1136 #endif
1137
1138         return f;
1139 }
1140
1141 /**
1142  * \brief Split Face Make Edge (SFME)
1143  *
1144  * Takes as input two vertices in a single face. An edge is created which divides the original face
1145  * into two distinct regions. One of the regions is assigned to the original face and it is closed off.
1146  * The second region has a new face assigned to it.
1147  *
1148  * \par Examples:
1149  * <pre>
1150  *     Before:               After:
1151  *      +--------+           +--------+
1152  *      |        |           |        |
1153  *      |        |           |   f1   |
1154  *     v1   f1   v2          v1======v2
1155  *      |        |           |   f2   |
1156  *      |        |           |        |
1157  *      +--------+           +--------+
1158  * </pre>
1159  *
1160  * \note the input vertices can be part of the same edge. This will
1161  * result in a two edged face. This is desirable for advanced construction
1162  * tools and particularly essential for edge bevel. Because of this it is
1163  * up to the caller to decide what to do with the extra edge.
1164  *
1165  * \note If \a holes is NULL, then both faces will lose
1166  * all holes from the original face.  Also, you cannot split between
1167  * a hole vert and a boundary vert; that case is handled by higher-
1168  * level wrapping functions (when holes are fully implemented, anyway).
1169  *
1170  * \note that holes represents which holes goes to the new face, and of
1171  * course this requires removing them from the existing face first, since
1172  * you cannot have linked list links inside multiple lists.
1173  *
1174  * \return A BMFace pointer
1175  */
1176 BMFace *bmesh_sfme(BMesh *bm, BMFace *f, BMVert *v1, BMVert *v2,
1177                    BMLoop **r_l,
1178 #ifdef USE_BMESH_HOLES
1179                    ListBase *holes,
1180 #endif
1181                    BMEdge *example,
1182                    const short nodouble
1183                    )
1184 {
1185 #ifdef USE_BMESH_HOLES
1186         BMLoopList *lst, *lst2;
1187 #endif
1188
1189         BMFace *f2;
1190         BMLoop *l_iter, *l_first;
1191         BMLoop *v1loop = NULL, *v2loop = NULL, *f1loop = NULL, *f2loop = NULL;
1192         BMEdge *e;
1193         int i, len, f1len, f2len, first_loop_f1;
1194
1195         /* verify that v1 and v2 are in face */
1196         len = f->len;
1197         for (i = 0, l_iter = BM_FACE_FIRST_LOOP(f); i < len; i++, l_iter = l_iter->next) {
1198                 if (l_iter->v == v1) v1loop = l_iter;
1199                 else if (l_iter->v == v2) v2loop = l_iter;
1200         }
1201
1202         if (!v1loop || !v2loop) {
1203                 return NULL;
1204         }
1205
1206         /* allocate new edge between v1 and v2 */
1207         e = BM_edge_create(bm, v1, v2, example, nodouble);
1208
1209         f2 = bm_face_create__sfme(bm, f);
1210         f1loop = bm_loop_create(bm, v2, e, f, v2loop);
1211         f2loop = bm_loop_create(bm, v1, e, f2, v1loop);
1212
1213         f1loop->prev = v2loop->prev;
1214         f2loop->prev = v1loop->prev;
1215         v2loop->prev->next = f1loop;
1216         v1loop->prev->next = f2loop;
1217
1218         f1loop->next = v1loop;
1219         f2loop->next = v2loop;
1220         v1loop->prev = f1loop;
1221         v2loop->prev = f2loop;
1222
1223 #ifdef USE_BMESH_HOLES
1224         lst = f->loops.first;
1225         lst2 = f2->loops.first;
1226
1227         lst2->first = lst2->last = f2loop;
1228         lst->first = lst->last = f1loop;
1229 #else
1230         /* find which of the faces the original first loop is in */
1231         l_iter = l_first = f1loop;
1232         first_loop_f1 = 0;
1233         do {
1234                 if (l_iter == f->l_first)
1235                         first_loop_f1 = 1;
1236         } while ((l_iter = l_iter->next) != l_first);
1237
1238         if (first_loop_f1) {
1239                 /* original first loop was in f1, find a suitable first loop for f2
1240                  * which is as similar as possible to f1. the order matters for tools
1241                  * such as duplifaces. */
1242                 if (f->l_first->prev == f1loop)
1243                         f2->l_first = f2loop->prev;
1244                 else if (f->l_first->next == f1loop)
1245                         f2->l_first = f2loop->next;
1246                 else
1247                         f2->l_first = f2loop;
1248         }
1249         else {
1250                 /* original first loop was in f2, further do same as above */
1251                 f2->l_first = f->l_first;
1252
1253                 if (f->l_first->prev == f2loop)
1254                         f->l_first = f1loop->prev;
1255                 else if (f->l_first->next == f2loop)
1256                         f->l_first = f1loop->next;
1257                 else
1258                         f->l_first = f1loop;
1259         }
1260 #endif
1261
1262         /* validate both loop */
1263         /* I don't know how many loops are supposed to be in each face at this point! FIXME */
1264
1265         /* go through all of f2's loops and make sure they point to it properly */
1266         l_iter = l_first = BM_FACE_FIRST_LOOP(f2);
1267         f2len = 0;
1268         do {
1269                 l_iter->f = f2;
1270                 f2len++;
1271         } while ((l_iter = l_iter->next) != l_first);
1272
1273         /* link up the new loops into the new edges radial */
1274         bmesh_radial_append(e, f1loop);
1275         bmesh_radial_append(e, f2loop);
1276
1277         f2->len = f2len;
1278
1279         f1len = 0;
1280         l_iter = l_first = BM_FACE_FIRST_LOOP(f);
1281         do {
1282                 f1len++;
1283         } while ((l_iter = l_iter->next) != l_first);
1284
1285         f->len = f1len;
1286
1287         if (r_l) *r_l = f2loop;
1288
1289 #ifdef USE_BMESH_HOLES
1290         if (holes) {
1291                 BLI_movelisttolist(&f2->loops, holes);
1292         }
1293         else {
1294                 /* this code is not significant until holes actually work */
1295                 //printf("warning: call to split face euler without holes argument; holes will be tossed.\n");
1296                 for (lst = f->loops.last; lst != f->loops.first; lst = lst2) {
1297                         lst2 = lst->prev;
1298                         BLI_mempool_free(bm->looplistpool, lst);
1299                 }
1300         }
1301 #endif
1302
1303         BM_CHECK_ELEMENT(e);
1304         BM_CHECK_ELEMENT(f);
1305         BM_CHECK_ELEMENT(f2);
1306         
1307         return f2;
1308 }
1309
1310 /**
1311  * \brief Split Edge Make Vert (SEMV)
1312  *
1313  * Takes \a e edge and splits it into two, creating a new vert.
1314  * \a tv should be one end of \a e : the newly created edge
1315  * will be attached to that end and is returned in \a r_e.
1316  *
1317  * \par Examples:
1318  * <pre>
1319  *                     E
1320  *     Before: OV-------------TV
1321  *
1322  *                 E       RE
1323  *     After:  OV------NV-----TV
1324  * </pre>
1325  *
1326  * \return The newly created BMVert pointer.
1327  */
1328 BMVert *bmesh_semv(BMesh *bm, BMVert *tv, BMEdge *e, BMEdge **r_e)
1329 {
1330         BMLoop *nextl;
1331         BMEdge *ne;
1332         BMVert *nv, *ov;
1333         int i, edok, valence1 = 0, valence2 = 0;
1334
1335         BLI_assert(bmesh_vert_in_edge(e, tv) != FALSE);
1336
1337         ov = bmesh_edge_other_vert_get(e, tv);
1338
1339         valence1 = bmesh_disk_count(ov);
1340
1341         valence2 = bmesh_disk_count(tv);
1342
1343         nv = BM_vert_create(bm, tv->co, tv);
1344         ne = BM_edge_create(bm, nv, tv, e, FALSE);
1345
1346         bmesh_disk_edge_remove(ne, tv);
1347         bmesh_disk_edge_remove(ne, nv);
1348
1349         /* remove e from tv's disk cycle */
1350         bmesh_disk_edge_remove(e, tv);
1351
1352         /* swap out tv for nv in e */
1353         bmesh_edge_swapverts(e, tv, nv);
1354
1355         /* add e to nv's disk cycle */
1356         bmesh_disk_edge_append(e, nv);
1357
1358         /* add ne to nv's disk cycle */
1359         bmesh_disk_edge_append(ne, nv);
1360
1361         /* add ne to tv's disk cycle */
1362         bmesh_disk_edge_append(ne, tv);
1363
1364         /* verify disk cycle */
1365         edok = bmesh_disk_validate(valence1, ov->e, ov);
1366         BMESH_ASSERT(edok != FALSE);
1367         edok = bmesh_disk_validate(valence2, tv->e, tv);
1368         BMESH_ASSERT(edok != FALSE);
1369         edok = bmesh_disk_validate(2, nv->e, nv);
1370         BMESH_ASSERT(edok != FALSE);
1371
1372         /* Split the radial cycle if present */
1373         nextl = e->l;
1374         e->l = NULL;
1375         if (nextl) {
1376                 BMLoop *nl, *l;
1377                 int radlen = bmesh_radial_length(nextl);
1378                 int first1 = 0, first2 = 0;
1379
1380                 /* Take the next loop. Remove it from radial. Split it. Append to appropriate radials */
1381                 while (nextl) {
1382                         l = nextl;
1383                         l->f->len++;
1384                         nextl = nextl != nextl->radial_next ? nextl->radial_next : NULL;
1385                         bmesh_radial_loop_remove(l, NULL);
1386
1387                         nl = bm_loop_create(bm, NULL, NULL, l->f, l);
1388                         nl->prev = l;
1389                         nl->next = (l->next);
1390                         nl->prev->next = nl;
1391                         nl->next->prev = nl;
1392                         nl->v = nv;
1393
1394                         /* assign the correct edge to the correct loop */
1395                         if (bmesh_verts_in_edge(nl->v, nl->next->v, e)) {
1396                                 nl->e = e;
1397                                 l->e = ne;
1398
1399                                 /* append l into ne's rad cycle */
1400                                 if (!first1) {
1401                                         first1 = 1;
1402                                         l->radial_next = l->radial_prev = NULL;
1403                                 }
1404
1405                                 if (!first2) {
1406                                         first2 = 1;
1407                                         l->radial_next = l->radial_prev = NULL;
1408                                 }
1409                                 
1410                                 bmesh_radial_append(nl->e, nl);
1411                                 bmesh_radial_append(l->e, l);
1412                         }
1413                         else if (bmesh_verts_in_edge(nl->v, nl->next->v, ne)) {
1414                                 nl->e = ne;
1415                                 l->e = e;
1416
1417                                 /* append l into ne's rad cycle */
1418                                 if (!first1) {
1419                                         first1 = 1;
1420                                         l->radial_next = l->radial_prev = NULL;
1421                                 }
1422
1423                                 if (!first2) {
1424                                         first2 = 1;
1425                                         l->radial_next = l->radial_prev = NULL;
1426                                 }
1427
1428                                 bmesh_radial_append(nl->e, nl);
1429                                 bmesh_radial_append(l->e, l);
1430                         }
1431
1432                 }
1433
1434                 /* verify length of radial cycle */
1435                 edok = bmesh_radial_validate(radlen, e->l);
1436                 BMESH_ASSERT(edok != FALSE);
1437                 edok = bmesh_radial_validate(radlen, ne->l);
1438                 BMESH_ASSERT(edok != FALSE);
1439
1440                 /* verify loop->v and loop->next->v pointers for e */
1441                 for (i = 0, l = e->l; i < radlen; i++, l = l->radial_next) {
1442                         BMESH_ASSERT(l->e == e);
1443                         //BMESH_ASSERT(l->radial_next == l);
1444                         BMESH_ASSERT(!(l->prev->e != ne && l->next->e != ne));
1445
1446                         edok = bmesh_verts_in_edge(l->v, l->next->v, e);
1447                         BMESH_ASSERT(edok != FALSE);
1448                         BMESH_ASSERT(l->v != l->next->v);
1449                         BMESH_ASSERT(l->e != l->next->e);
1450
1451                         /* verify loop cycle for kloop-> */
1452                         BM_CHECK_ELEMENT(l);
1453                         BM_CHECK_ELEMENT(l->v);
1454                         BM_CHECK_ELEMENT(l->e);
1455                         BM_CHECK_ELEMENT(l->f);
1456                 }
1457                 /* verify loop->v and loop->next->v pointers for ne */
1458                 for (i = 0, l = ne->l; i < radlen; i++, l = l->radial_next) {
1459                         BMESH_ASSERT(l->e == ne);
1460                         // BMESH_ASSERT(l->radial_next == l);
1461                         BMESH_ASSERT(!(l->prev->e != e && l->next->e != e));
1462                         edok = bmesh_verts_in_edge(l->v, l->next->v, ne);
1463                         BMESH_ASSERT(edok != FALSE);
1464                         BMESH_ASSERT(l->v != l->next->v);
1465                         BMESH_ASSERT(l->e != l->next->e);
1466
1467                         BM_CHECK_ELEMENT(l);
1468                         BM_CHECK_ELEMENT(l->v);
1469                         BM_CHECK_ELEMENT(l->e);
1470                         BM_CHECK_ELEMENT(l->f);
1471                 }
1472         }
1473
1474         BM_CHECK_ELEMENT(ne);
1475         BM_CHECK_ELEMENT(nv);
1476         BM_CHECK_ELEMENT(ov);
1477         BM_CHECK_ELEMENT(e);
1478         BM_CHECK_ELEMENT(tv);
1479
1480         if (r_e) *r_e = ne;
1481         return nv;
1482 }
1483
1484 /**
1485  * \brief Join Edge Kill Vert (JEKV)
1486  *
1487  * Takes an edge \a ke and pointer to one of its vertices \a kv
1488  * and collapses the edge on that vertex.
1489  *
1490  * \par Examples:
1491  *
1492  * <pre>
1493  *     Before:         OE      KE
1494  *                   ------- -------
1495  *                   |     ||      |
1496  *                  OV     KV      TV
1497  *
1498  *
1499  *     After:              OE
1500  *                   ---------------
1501  *                   |             |
1502  *                  OV             TV
1503  * </pre>
1504  *
1505  * \par Restrictions:
1506  * KV is a vertex that must have a valance of exactly two. Furthermore
1507  * both edges in KV's disk cycle (OE and KE) must be unique (no double edges).
1508  *
1509  * \return The resulting edge, NULL for failure.
1510  *
1511  * \note This euler has the possibility of creating
1512  * faces with just 2 edges. It is up to the caller to decide what to do with
1513  * these faces.
1514  */
1515 BMEdge *bmesh_jekv(BMesh *bm, BMEdge *ke, BMVert *kv, const short check_edge_double)
1516 {
1517         BMEdge *oe;
1518         BMVert *ov, *tv;
1519         BMLoop *killoop, *l;
1520         int len, radlen = 0, halt = 0, i, valence1, valence2, edok;
1521
1522         if (bmesh_vert_in_edge(ke, kv) == 0) {
1523                 return NULL;
1524         }
1525
1526         len = bmesh_disk_count(kv);
1527         
1528         if (len == 2) {
1529                 oe = bmesh_disk_edge_next(ke, kv);
1530                 tv = bmesh_edge_other_vert_get(ke, kv);
1531                 ov = bmesh_edge_other_vert_get(oe, kv);
1532                 halt = bmesh_verts_in_edge(kv, tv, oe); /* check for double edge */
1533                 
1534                 if (halt) {
1535                         return NULL;
1536                 }
1537                 else {
1538                         BMEdge *e_splice;
1539
1540                         /* For verification later, count valence of ov and t */
1541                         valence1 = bmesh_disk_count(ov);
1542                         valence2 = bmesh_disk_count(tv);
1543
1544                         if (check_edge_double) {
1545                                 e_splice = BM_edge_exists(tv, ov);
1546                         }
1547
1548                         /* remove oe from kv's disk cycle */
1549                         bmesh_disk_edge_remove(oe, kv);
1550                         /* relink oe->kv to be oe->tv */
1551                         bmesh_edge_swapverts(oe, kv, tv);
1552                         /* append oe to tv's disk cycle */
1553                         bmesh_disk_edge_append(oe, tv);
1554                         /* remove ke from tv's disk cycle */
1555                         bmesh_disk_edge_remove(ke, tv);
1556
1557                         /* deal with radial cycle of ke */
1558                         radlen = bmesh_radial_length(ke->l);
1559                         if (ke->l) {
1560                                 /* first step, fix the neighboring loops of all loops in ke's radial cycle */
1561                                 for (i = 0, killoop = ke->l; i < radlen; i++, killoop = killoop->radial_next) {
1562                                         /* relink loops and fix vertex pointer */
1563                                         if (killoop->next->v == kv) {
1564                                                 killoop->next->v = tv;
1565                                         }
1566
1567                                         killoop->next->prev = killoop->prev;
1568                                         killoop->prev->next = killoop->next;
1569                                         if (BM_FACE_FIRST_LOOP(killoop->f) == killoop) {
1570                                                 BM_FACE_FIRST_LOOP(killoop->f) = killoop->next;
1571                                         }
1572                                         killoop->next = NULL;
1573                                         killoop->prev = NULL;
1574
1575                                         /* fix len attribute of face */
1576                                         killoop->f->len--;
1577                                 }
1578                                 /* second step, remove all the hanging loops attached to ke */
1579                                 radlen = bmesh_radial_length(ke->l);
1580
1581                                 if (LIKELY(radlen)) {
1582                                         BMLoop **loops = NULL;
1583                                         BLI_array_fixedstack_declare(loops, BM_NGON_STACK_SIZE, radlen, __func__);
1584
1585                                         killoop = ke->l;
1586
1587                                         /* this should be wrapped into a bme_free_radial function to be used by bmesh_KF as well... */
1588                                         for (i = 0; i < radlen; i++) {
1589                                                 loops[i] = killoop;
1590                                                 killoop = killoop->radial_next;
1591                                         }
1592                                         for (i = 0; i < radlen; i++) {
1593                                                 bm->totloop--;
1594                                                 BLI_mempool_free(bm->lpool, loops[i]);
1595                                         }
1596                                         BLI_array_fixedstack_free(loops);
1597                                 }
1598
1599                                 /* Validate radial cycle of oe */
1600                                 edok = bmesh_radial_validate(radlen, oe->l);
1601                                 BMESH_ASSERT(edok != FALSE);
1602                         }
1603
1604                         /* deallocate edg */
1605                         bm_kill_only_edge(bm, ke);
1606
1607                         /* deallocate verte */
1608                         bm_kill_only_vert(bm, kv);
1609
1610                         /* Validate disk cycle lengths of ov, tv are unchanged */
1611                         edok = bmesh_disk_validate(valence1, ov->e, ov);
1612                         BMESH_ASSERT(edok != FALSE);
1613                         edok = bmesh_disk_validate(valence2, tv->e, tv);
1614                         BMESH_ASSERT(edok != FALSE);
1615
1616                         /* Validate loop cycle of all faces attached to oe */
1617                         for (i = 0, l = oe->l; i < radlen; i++, l = l->radial_next) {
1618                                 BMESH_ASSERT(l->e == oe);
1619                                 edok = bmesh_verts_in_edge(l->v, l->next->v, oe);
1620                                 BMESH_ASSERT(edok != FALSE);
1621                                 edok = bmesh_loop_validate(l->f);
1622                                 BMESH_ASSERT(edok != FALSE);
1623
1624                                 BM_CHECK_ELEMENT(l);
1625                                 BM_CHECK_ELEMENT(l->v);
1626                                 BM_CHECK_ELEMENT(l->e);
1627                                 BM_CHECK_ELEMENT(l->f);
1628                         }
1629
1630                         if (check_edge_double) {
1631                                 if (e_splice) {
1632                                         /* removes e_splice */
1633                                         BM_edge_splice(bm, e_splice, oe);
1634                                 }
1635                         }
1636
1637                         BM_CHECK_ELEMENT(ov);
1638                         BM_CHECK_ELEMENT(tv);
1639                         BM_CHECK_ELEMENT(oe);
1640
1641                         return oe;
1642                 }
1643         }
1644         return NULL;
1645 }
1646
1647 /**
1648  * \brief Join Face Kill Edge (JFKE)
1649  *
1650  * Takes two faces joined by a single 2-manifold edge and fuses them together.
1651  * The edge shared by the faces must not be connected to any other edges which have
1652  * Both faces in its radial cycle
1653  *
1654  * \par Examples:
1655  * <pre>
1656  *           A                   B
1657  *      +--------+           +--------+
1658  *      |        |           |        |
1659  *      |   f1   |           |   f1   |
1660  *     v1========v2 = Ok!    v1==V2==v3 == Wrong!
1661  *      |   f2   |           |   f2   |
1662  *      |        |           |        |
1663  *      +--------+           +--------+
1664  * </pre>
1665  *
1666  * In the example A, faces \a f1 and \a f2 are joined by a single edge,
1667  * and the euler can safely be used.
1668  * In example B however, \a f1 and \a f2 are joined by multiple edges and will produce an error.
1669  * The caller in this case should call #bmesh_jekv on the extra edges
1670  * before attempting to fuse \a f1 and \a f2.
1671  *
1672  * \note The order of arguments decides whether or not certain per-face attributes are present
1673  * in the resultant face. For instance vertex winding, material index, smooth flags, etc are inherited
1674  * from \a f1, not \a f2.
1675  *
1676  * \return A BMFace pointer
1677  */
1678 BMFace *bmesh_jfke(BMesh *bm, BMFace *f1, BMFace *f2, BMEdge *e)
1679 {
1680         BMLoop *l_iter, *f1loop = NULL, *f2loop = NULL;
1681         int newlen = 0, i, f1len = 0, f2len = 0, edok;
1682
1683         /* can't join a face to itself */
1684         if (f1 == f2) {
1685                 return NULL;
1686         }
1687
1688         /* validate that edge is 2-manifold edge */
1689         if (!BM_edge_is_manifold(e)) {
1690                 return NULL;
1691         }
1692
1693         /* verify that e is in both f1 and f2 */
1694         f1len = f1->len;
1695         f2len = f2->len;
1696
1697         if (!((f1loop = BM_face_edge_share_loop(f1, e)) &&
1698               (f2loop = BM_face_edge_share_loop(f2, e))))
1699         {
1700                 return NULL;
1701         }
1702
1703         /* validate direction of f2's loop cycle is compatible */
1704         if (f1loop->v == f2loop->v) {
1705                 return NULL;
1706         }
1707
1708         /* validate that for each face, each vertex has another edge in its disk cycle that is
1709          * not e, and not shared. */
1710         if (bmesh_radial_face_find(f1loop->next->e, f2) ||
1711             bmesh_radial_face_find(f1loop->prev->e, f2) ||
1712             bmesh_radial_face_find(f2loop->next->e, f1) ||
1713             bmesh_radial_face_find(f2loop->prev->e, f1) )
1714         {
1715                 return NULL;
1716         }
1717
1718         /* validate only one shared edge */
1719         if (BM_face_share_edge_count(f1, f2) > 1) {
1720                 return NULL;
1721         }
1722
1723         /* validate no internal join */
1724         for (i = 0, l_iter = BM_FACE_FIRST_LOOP(f1); i < f1len; i++, l_iter = l_iter->next) {
1725                 BM_elem_flag_disable(l_iter->v, BM_ELEM_INTERNAL_TAG);
1726         }
1727         for (i = 0, l_iter = BM_FACE_FIRST_LOOP(f2); i < f2len; i++, l_iter = l_iter->next) {
1728                 BM_elem_flag_disable(l_iter->v, BM_ELEM_INTERNAL_TAG);
1729         }
1730
1731         for (i = 0, l_iter = BM_FACE_FIRST_LOOP(f1); i < f1len; i++, l_iter = l_iter->next) {
1732                 if (l_iter != f1loop) {
1733                         BM_elem_flag_enable(l_iter->v, BM_ELEM_INTERNAL_TAG);
1734                 }
1735         }
1736         for (i = 0, l_iter = BM_FACE_FIRST_LOOP(f2); i < f2len; i++, l_iter = l_iter->next) {
1737                 if (l_iter != f2loop) {
1738                         /* as soon as a duplicate is found, bail out */
1739                         if (BM_elem_flag_test(l_iter->v, BM_ELEM_INTERNAL_TAG)) {
1740                                 return NULL;
1741                         }
1742                 }
1743         }
1744
1745         /* join the two loop */
1746         f1loop->prev->next = f2loop->next;
1747         f2loop->next->prev = f1loop->prev;
1748         
1749         f1loop->next->prev = f2loop->prev;
1750         f2loop->prev->next = f1loop->next;
1751         
1752         /* if f1loop was baseloop, make f1loop->next the base. */
1753         if (BM_FACE_FIRST_LOOP(f1) == f1loop)
1754                 BM_FACE_FIRST_LOOP(f1) = f1loop->next;
1755
1756         /* increase length of f1 */
1757         f1->len += (f2->len - 2);
1758
1759         /* make sure each loop points to the proper face */
1760         newlen = f1->len;
1761         for (i = 0, l_iter = BM_FACE_FIRST_LOOP(f1); i < newlen; i++, l_iter = l_iter->next)
1762                 l_iter->f = f1;
1763         
1764         /* remove edge from the disk cycle of its two vertices */
1765         bmesh_disk_edge_remove(f1loop->e, f1loop->e->v1);
1766         bmesh_disk_edge_remove(f1loop->e, f1loop->e->v2);
1767         
1768         /* deallocate edge and its two loops as well as f2 */
1769         BLI_mempool_free(bm->toolflagpool, f1loop->e->oflags);
1770         BLI_mempool_free(bm->epool, f1loop->e);
1771         bm->totedge--;
1772         BLI_mempool_free(bm->lpool, f1loop);
1773         bm->totloop--;
1774         BLI_mempool_free(bm->lpool, f2loop);
1775         bm->totloop--;
1776         BLI_mempool_free(bm->toolflagpool, f2->oflags);
1777         BLI_mempool_free(bm->fpool, f2);
1778         bm->totface--;
1779         /* account for both above */
1780         bm->elem_index_dirty |= BM_EDGE | BM_FACE;
1781
1782         BM_CHECK_ELEMENT(f1);
1783
1784         /* validate the new loop cycle */
1785         edok = bmesh_loop_validate(f1);
1786         BMESH_ASSERT(edok != FALSE);
1787         
1788         return f1;
1789 }
1790
1791 /**
1792  * \brief Splice Vert
1793  *
1794  * Merges two verts into one (\a v into \a vtarget).
1795  *
1796  * \return Success
1797  */
1798 int BM_vert_splice(BMesh *bm, BMVert *v, BMVert *vtarget)
1799 {
1800         BMEdge *e;
1801         BMLoop *l;
1802         BMIter liter;
1803
1804         /* verts already spliced */
1805         if (v == vtarget) {
1806                 return FALSE;
1807         }
1808
1809         /* retarget all the loops of v to vtarget */
1810         BM_ITER_ELEM (l, &liter, v, BM_LOOPS_OF_VERT) {
1811                 l->v = vtarget;
1812         }
1813
1814         /* move all the edges from v's disk to vtarget's disk */
1815         while ((e = v->e)) {
1816                 bmesh_disk_edge_remove(e, v);
1817                 bmesh_edge_swapverts(e, v, vtarget);
1818                 bmesh_disk_edge_append(e, vtarget);
1819         }
1820
1821         BM_CHECK_ELEMENT(v);
1822         BM_CHECK_ELEMENT(vtarget);
1823
1824         /* v is unused now, and can be killed */
1825         BM_vert_kill(bm, v);
1826
1827         return TRUE;
1828 }
1829
1830 /**
1831  * \brief Separate Vert
1832  *
1833  * Separates all disjoint fans that meet at a vertex, making a unique
1834  * vertex for each region. returns an array of all resulting vertices.
1835  *
1836  * \note this is a low level function, bm_edge_separate needs to run on edges first
1837  * or, the faces sharing verts must not be sharing edges for them to split at least.
1838  *
1839  * \return Success
1840  */
1841 int bmesh_vert_separate(BMesh *bm, BMVert *v, BMVert ***r_vout, int *r_vout_len)
1842 {
1843         BMEdge **stack = NULL;
1844         BLI_array_declare(stack);
1845         BMVert **verts = NULL;
1846         GHash *visithash;
1847         BMIter eiter, liter;
1848         BMLoop *l;
1849         BMEdge *e;
1850         int i, maxindex;
1851         BMLoop *nl;
1852
1853         visithash = BLI_ghash_ptr_new(__func__);
1854
1855         maxindex = 0;
1856         BM_ITER_ELEM (e, &eiter, v, BM_EDGES_OF_VERT) {
1857                 if (BLI_ghash_haskey(visithash, e)) {
1858                         continue;
1859                 }
1860
1861                 /* Prime the stack with this unvisited edge */
1862                 BLI_array_append(stack, e);
1863
1864                 /* Considering only edges and faces incident on vertex v, walk
1865                  * the edges & faces and assign an index to each connected set */
1866                 while ((e = BLI_array_pop(stack))) {
1867                         BLI_ghash_insert(visithash, e, SET_INT_IN_POINTER(maxindex));
1868
1869                         BM_ITER_ELEM (l, &liter, e, BM_LOOPS_OF_EDGE) {
1870                                 nl = (l->v == v) ? l->prev : l->next;
1871                                 if (!BLI_ghash_haskey(visithash, nl->e)) {
1872                                         BLI_array_append(stack, nl->e);
1873                                 }
1874                         }
1875                 }
1876
1877                 maxindex++;
1878         }
1879
1880         /* Make enough verts to split v for each group */
1881         verts = MEM_callocN(sizeof(BMVert *) * maxindex, __func__);
1882         verts[0] = v;
1883         for (i = 1; i < maxindex; i++) {
1884                 verts[i] = BM_vert_create(bm, v->co, v);
1885         }
1886
1887         /* Replace v with the new verts in each group */
1888 #if 0
1889         BM_ITER_ELEM (l, &liter, v, BM_LOOPS_OF_VERT) {
1890                 /* call first since its faster then a hash lookup */
1891                 if (l->v != v) {
1892                         continue;
1893                 }
1894                 i = GET_INT_FROM_POINTER(BLI_ghash_lookup(visithash, l->e));
1895                 if (i == 0) {
1896                         continue;
1897                 }
1898
1899                 /* Loops here should always refer to an edge that has v as an
1900                  * endpoint. For each appearance of this vert in a face, there
1901                  * will actually be two iterations: one for the loop heading
1902                  * towards vertex v, and another for the loop heading out from
1903                  * vertex v. Only need to swap the vertex on one of those times,
1904                  * on the outgoing loop. */
1905
1906                 /* XXX - because this clobbers the iterator, this *whole* block is commented, see below */
1907                 l->v = verts[i];
1908         }
1909 #else
1910         /* note: this is the same as the commented code above *except* that it doesn't break iterator
1911          * by modifying data it loops over [#30632], this re-uses the 'stack' variable which is a bit
1912          * bad practice but save alloc'ing a new array - note, the comment above is useful, keep it
1913          * if you are tidying up code - campbell */
1914         BLI_array_empty(stack);
1915         BM_ITER_ELEM (l, &liter, v, BM_LOOPS_OF_VERT) {
1916                 if (l->v == v) {
1917                         BLI_array_append(stack, (BMEdge *)l);
1918                 }
1919         }
1920         while ((l = (BMLoop *)(BLI_array_pop(stack)))) {
1921                 if ((i = GET_INT_FROM_POINTER(BLI_ghash_lookup(visithash, l->e)))) {
1922                         l->v = verts[i];
1923                 }
1924         }
1925 #endif
1926
1927         BLI_array_free(stack);
1928
1929         BM_ITER_ELEM (e, &eiter, v, BM_EDGES_OF_VERT) {
1930                 i = GET_INT_FROM_POINTER(BLI_ghash_lookup(visithash, e));
1931                 if (i == 0) {
1932                         continue;
1933                 }
1934
1935                 BLI_assert(e->v1 == v || e->v2 == v);
1936                 bmesh_disk_edge_remove(e, v);
1937                 bmesh_edge_swapverts(e, v, verts[i]);
1938                 bmesh_disk_edge_append(e, verts[i]);
1939         }
1940
1941         BLI_ghash_free(visithash, NULL, NULL);
1942
1943         for (i = 0; i < maxindex; i++) {
1944                 BM_CHECK_ELEMENT(verts[i]);
1945         }
1946
1947         if (r_vout_len != NULL) {
1948                 *r_vout_len = maxindex;
1949         }
1950
1951         if (r_vout != NULL) {
1952                 *r_vout = verts;
1953         }
1954         else {
1955                 MEM_freeN(verts);
1956         }
1957
1958         return TRUE;
1959 }
1960
1961 /**
1962  * High level function which wraps both #bmesh_vert_separate and #bmesh_edge_separate
1963  */
1964 int BM_vert_separate(BMesh *bm, BMVert *v, BMVert ***r_vout, int *r_vout_len,
1965                      BMEdge **e_in, int e_in_len)
1966 {
1967         int i;
1968
1969         for (i = 0; i < e_in_len; i++) {
1970                 BMEdge *e = e_in[i];
1971                 if (e->l && BM_vert_in_edge(e, v)) {
1972                         bmesh_edge_separate(bm, e, e->l);
1973                 }
1974         }
1975
1976         return bmesh_vert_separate(bm, v, r_vout, r_vout_len);
1977 }
1978
1979 /**
1980  * \brief Splice Edge
1981  *
1982  * Splice two unique edges which share the same two vertices into one edge.
1983  *
1984  * \return Success
1985  *
1986  * \note Edges must already have the same vertices.
1987  */
1988 int BM_edge_splice(BMesh *bm, BMEdge *e, BMEdge *etarget)
1989 {
1990         BMLoop *l;
1991
1992         if (!BM_vert_in_edge(e, etarget->v1) || !BM_vert_in_edge(e, etarget->v2)) {
1993                 /* not the same vertices can't splice */
1994                 return FALSE;
1995         }
1996
1997         while (e->l) {
1998                 l = e->l;
1999                 BLI_assert(BM_vert_in_edge(etarget, l->v));
2000                 BLI_assert(BM_vert_in_edge(etarget, l->next->v));
2001                 bmesh_radial_loop_remove(l, e);
2002                 bmesh_radial_append(etarget, l);
2003         }
2004
2005         BLI_assert(bmesh_radial_length(e->l) == 0);
2006
2007         BM_CHECK_ELEMENT(e);
2008         BM_CHECK_ELEMENT(etarget);
2009
2010         /* removes from disks too */
2011         BM_edge_kill(bm, e);
2012
2013         return TRUE;
2014 }
2015
2016 /**
2017  * \brief Separate Edge
2018  *
2019  * Separates a single edge into two edge: the original edge and
2020  * a new edge that has only \a l_sep in its radial.
2021  *
2022  * \return Success
2023  *
2024  * \note Does nothing if \a l_sep is already the only loop in the
2025  * edge radial.
2026  */
2027 int bmesh_edge_separate(BMesh *bm, BMEdge *e, BMLoop *l_sep)
2028 {
2029         BMEdge *ne;
2030         int radlen;
2031
2032         BLI_assert(l_sep->e == e);
2033         BLI_assert(e->l);
2034         
2035         radlen = bmesh_radial_length(e->l);
2036         if (radlen < 2) {
2037                 /* no cut required */
2038                 return TRUE;
2039         }
2040
2041         if (l_sep == e->l) {
2042                 e->l = l_sep->radial_next;
2043         }
2044
2045         ne = BM_edge_create(bm, e->v1, e->v2, e, FALSE);
2046         bmesh_radial_loop_remove(l_sep, e);
2047         bmesh_radial_append(ne, l_sep);
2048         l_sep->e = ne;
2049
2050         BLI_assert(bmesh_radial_length(e->l) == radlen - 1);
2051         BLI_assert(bmesh_radial_length(ne->l) == 1);
2052
2053         BM_CHECK_ELEMENT(ne);
2054         BM_CHECK_ELEMENT(e);
2055
2056         return TRUE;
2057 }
2058
2059 /**
2060  * \brief Unglue Region Make Vert (URMV)
2061  *
2062  * Disconnects a face from its vertex fan at loop \a sl
2063  *
2064  * \return The newly created BMVert
2065  */
2066 BMVert *bmesh_urmv_loop(BMesh *bm, BMLoop *sl)
2067 {
2068         BMVert **vtar;
2069         int len, i;
2070         BMVert *nv = NULL;
2071         BMVert *sv = sl->v;
2072
2073         /* peel the face from the edge radials on both sides of the
2074          * loop vert, disconnecting the face from its fan */
2075         bmesh_edge_separate(bm, sl->e, sl);
2076         bmesh_edge_separate(bm, sl->prev->e, sl->prev);
2077
2078         if (bmesh_disk_count(sv) == 2) {
2079                 /* If there are still only two edges out of sv, then
2080                  * this whole URMV was just a no-op, so exit now. */
2081                 return sv;
2082         }
2083
2084         /* Update the disk start, so that v->e points to an edge
2085          * not touching the split loop. This is so that BM_vert_split
2086          * will leave the original sv on some *other* fan (not the
2087          * one-face fan that holds the unglue face). */
2088         while (sv->e == sl->e || sv->e == sl->prev->e) {
2089                 sv->e = bmesh_disk_edge_next(sv->e, sv);
2090         }
2091
2092         /* Split all fans connected to the vert, duplicating it for
2093          * each fans. */
2094         bmesh_vert_separate(bm, sv, &vtar, &len);
2095
2096         /* There should have been at least two fans cut apart here,
2097          * otherwise the early exit would have kicked in. */
2098         BLI_assert(len >= 2);
2099
2100         nv = sl->v;
2101
2102         /* Desired result here is that a new vert should always be
2103          * created for the unglue face. This is so we can glue any
2104          * extras back into the original vert. */
2105         BLI_assert(nv != sv);
2106         BLI_assert(sv == vtar[0]);
2107
2108         /* If there are more than two verts as a result, glue together
2109          * all the verts except the one this URMV intended to create */
2110         if (len > 2) {
2111                 for (i = 0; i < len; i++) {
2112                         if (vtar[i] == nv) {
2113                                 break;
2114                         }
2115                 }
2116
2117                 if (i != len) {
2118                         /* Swap the single vert that was needed for the
2119                          * unglue into the last array slot */
2120                         SWAP(BMVert *, vtar[i], vtar[len - 1]);
2121
2122                         /* And then glue the rest back together */
2123                         for (i = 1; i < len - 1; i++) {
2124                                 BM_vert_splice(bm, vtar[i], vtar[0]);
2125                         }
2126                 }
2127         }
2128
2129         MEM_freeN(vtar);
2130
2131         return nv;
2132 }
2133
2134 /**
2135  * \brief Unglue Region Make Vert (URMV)
2136  *
2137  * Disconnects sf from the vertex fan at \a sv
2138  *
2139  * \return The newly created BMVert
2140  */
2141 BMVert *bmesh_urmv(BMesh *bm, BMFace *sf, BMVert *sv)
2142 {
2143         BMLoop *l = BM_face_vert_share_loop(sf, sv);
2144         return bmesh_urmv_loop(bm, l);
2145 }