Revision: 18622
          
http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=18622
Author:   joeedh
Date:     2009-01-22 14:59:30 +0100 (Thu, 22 Jan 2009)

Log Message:
-----------
destruction of previous slot api. if it returns, it'll
be in the form of functions that return pointer
references to the slot data or something.

Modified Paths:
--------------
    branches/bmesh/blender/source/blender/bmesh/bmesh.h
    branches/bmesh/blender/source/blender/bmesh/bmesh_operators.h
    branches/bmesh/blender/source/blender/bmesh/intern/bmesh_operators.c
    branches/bmesh/blender/source/blender/bmesh/operators/dissolveops.c
    branches/bmesh/blender/source/blender/bmesh/operators/subdivideop.c
    branches/bmesh/blender/source/blender/bmesh/operators/triangulateop.c

Modified: branches/bmesh/blender/source/blender/bmesh/bmesh.h
===================================================================
--- branches/bmesh/blender/source/blender/bmesh/bmesh.h 2009-01-22 12:34:52 UTC 
(rev 18621)
+++ branches/bmesh/blender/source/blender/bmesh/bmesh.h 2009-01-22 13:59:30 UTC 
(rev 18622)
@@ -107,7 +107,6 @@
        void *data;
 } BMNode;
 
-struct bmop_error;
 typedef struct BMesh {
        ListBase verts, edges, polys;
        struct BLI_mempool *vpool;
@@ -126,7 +125,7 @@
        struct BLI_mempool *flagpool;                                   
/*memory pool for dynamically allocated flag layers*/
        int stackdepth;                                                         
        /*current depth of operator stack*/
        int totflags, walkers;                                                  
/*total number of tool flag layers*/
-       ListBase errorstack; /*privately used by the operator error reporting 
system*/
+       ListBase errorstack;
 } BMesh;
 
 typedef struct BMVert {        

Modified: branches/bmesh/blender/source/blender/bmesh/bmesh_operators.h
===================================================================
--- branches/bmesh/blender/source/blender/bmesh/bmesh_operators.h       
2009-01-22 12:34:52 UTC (rev 18621)
+++ branches/bmesh/blender/source/blender/bmesh/bmesh_operators.h       
2009-01-22 13:59:30 UTC (rev 18622)
@@ -30,30 +30,6 @@
        } data;
 }BMOpSlot;
 
-/*these macros are used for iterating over slot buffers.
-  for example:
-  int i;
-
-  for (ptr=BMOS_IterNewP(i, slot); ptr; ptr=BMOS_IterStepP(i, slot)) {
-  }
-
-  int ival;
-  for (ival=BMOS_IterNewI(i, slot); !BMOS_IterDoneI(i, slot); 
ival=BMOS_IterStepI(i, slot) {
-  }
-*/
-/*remember, the ',' operator executes all expressions seperated by ','
-  (left to right) but uses the value of the right-most one.*/
-#define BMOS_IterNewP(stateint, slot)  (stateint = 0, slot->len>0 ? 
*(void**)slot->data.p : NULL)
-#define BMOS_IterStepP(stateint, slot) (stateint++,stateint>=slot->len ? NULL 
: ((void**)slot->data.buf)[stateint])
-
-#define BMOS_IterNewF(stateint, slot)  (stateint = 0, slot->len>0 ? 
*(float*)slot->data.p : NULL)
-#define BMOS_IterDoneF(stateint, slot) (stateint >= slot->len)
-#define BMOS_IterStepF(stateint, slot) (stateint++,stateint>=slot->len ? NULL 
: ((float*)slot->data.buf)[stateint])
-
-#define BMOS_IterNewI(stateint, slot)  (stateint = 0, slot->len>0 ? 
*(int*)slot->data.p : NULL)
-#define BMOS_IterDoneI(stateint, slot) (stateint >= slot->len)
-#define BMOS_IterStepI(stateint, slot) (stateint++,stateint>=slot->len ? NULL 
: ((int*)slot->data.buf)[stateint])
-
 /*operators represent logical, executable mesh modules.*/
 #define BMOP_MAX_SLOTS                 16              /*way more than 
probably needed*/
 
@@ -108,15 +84,13 @@
 /*------ error code defines -------*/
 
 /*error messages*/
-#define BMERR_SELF_INTERSECTING        1
+#define BMERR_SELF_INTERSECTING        1
 
 static char *bmop_error_messages[] = {
-       0,
-       "Self intersection error",
+       0,
+       "Self intersection error",
 };
 
-#define BMERR_TOTAL (sizeof(error_messages) / sizeof(void*) - 1)
-
 /*------------begin operator defines (see bmesh_opdefines.c too)------------*/
 /*split op*/
 #define BMOP_SPLIT                             0

Modified: branches/bmesh/blender/source/blender/bmesh/intern/bmesh_operators.c
===================================================================
--- branches/bmesh/blender/source/blender/bmesh/intern/bmesh_operators.c        
2009-01-22 12:34:52 UTC (rev 18621)
+++ branches/bmesh/blender/source/blender/bmesh/intern/bmesh_operators.c        
2009-01-22 13:59:30 UTC (rev 18622)
@@ -2,7 +2,6 @@
 
 #include "BLI_memarena.h"
 #include "BLI_mempool.h"
-#include "BLI_blenlib.h"
 
 #include "BKE_utildefines.h"
 
@@ -32,46 +31,6 @@
        sizeof(void*)   /* pointer buffer */
 };
 
-/*error system*/
-typedef struct bmop_error {
-       struct bmop_error *next, *prev;
-       int errorcode;
-       char *msg;
-} bmop_error;
-
-void BMOP_RaiseError(BMesh *bm, int errcode, char *msg)
-{
-       bmop_error *err = MEM_callocN(sizeof(bmop_error), "bmop_error");
-       err->errorcode = errcode;
-       err->msg = msg;
-       BLI_addhead(&bm->errorstack, err);
-}
-
-/*returns error code or 0 if no error*/
-int BMOP_GetError(BMesh *bm, char **msg)
-{
-       bmop_error *err = bm->errorstack.first;
-       if (!err) return 0;
-       
-       if (msg) *msg = err->msg;
-
-       return err->errorcode;
-}
-
-int BMOP_CheckError(BMesh *bm)
-{
-       return bm->errorstack.first != NULL;
-}
-
-int BMOP_PopError(BMesh *bm, char **msg) 
-{
-       int errorcode = BMOP_GetError(bm, msg);
-       if (errorcode)
-               BLI_remlink(&bm->errorstack, &bm->errorstack.first);
-
-       return errorcode;
-}
-
 /*
  * BMESH OPSTACK PUSH
  *

Modified: branches/bmesh/blender/source/blender/bmesh/operators/dissolveops.c
===================================================================
--- branches/bmesh/blender/source/blender/bmesh/operators/dissolveops.c 
2009-01-22 12:34:52 UTC (rev 18621)
+++ branches/bmesh/blender/source/blender/bmesh/operators/dissolveops.c 
2009-01-22 13:59:30 UTC (rev 18622)
@@ -43,7 +43,8 @@
        }
        */
 
-       for (vert=BMOS_IterNewP(i, vinput); vert; vert = BMOS_IterStepP(i, 
vinput)) {
+       for (i=0; i<vinput->len; i++) {
+               vert = ((BMVert**)vinput->data.p)[i];
                BM_Dissolve_Disk(bmesh, vert);
        }
 }
\ No newline at end of file

Modified: branches/bmesh/blender/source/blender/bmesh/operators/subdivideop.c
===================================================================
--- branches/bmesh/blender/source/blender/bmesh/operators/subdivideop.c 
2009-01-22 12:34:52 UTC (rev 18621)
+++ branches/bmesh/blender/source/blender/bmesh/operators/subdivideop.c 
2009-01-22 13:59:30 UTC (rev 18622)
@@ -250,7 +250,8 @@
        einput = BMO_GetSlot(op, BMOP_ESUBDIVIDE_EDGES);
 
        /*first go through and split edges*/
-       for (edge=BMOS_IterNewP(i, einput); edge; edge = BMOS_IterStepP(i, 
einput)) {
+       for (i=0; i<einput->len; i++) {
+               edge = ((BMEdge**)einput->data.p)[i];
                v1 = BM_Split_Edge(bmesh, edge->v1, edge, &nedge, 0.5, 1);
                BMO_SetFlag(bmesh, v1, SUBD_SPLIT);
                BMO_SetFlag(bmesh, nedge, SUBD_SPLIT);

Modified: branches/bmesh/blender/source/blender/bmesh/operators/triangulateop.c
===================================================================
--- branches/bmesh/blender/source/blender/bmesh/operators/triangulateop.c       
2009-01-22 12:34:52 UTC (rev 18621)
+++ branches/bmesh/blender/source/blender/bmesh/operators/triangulateop.c       
2009-01-22 13:59:30 UTC (rev 18622)
@@ -19,8 +19,10 @@
        int i, count = 0;
        
        finput = BMO_GetSlot(op, BMOP_ESUBDIVIDE_EDGES);
-       
-       for (face=BMOS_IterNewP(i, finput); face; face=BMOS_IterStepP(i, 
finput)) {
+
+       for (i=0; i<finput->len; i++) {
+               face = ((BMFace**)finput->data.p)[i];
+
                /*HACK! need to discuss with Briggs why the function takes an 
                  externally-allocated array of vert coordinates in the first 
place.*/
                if (face->len > 400) projverts = 
MEM_callocN(sizeof(float)*3*face->len, "projverts");


_______________________________________________
Bf-blender-cvs mailing list
[email protected]
http://lists.blender.org/mailman/listinfo/bf-blender-cvs

Reply via email to