[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [15710] branches/soc-2008-mxcurioni/source /blender/freestyle: soc-2008-mxcurioni: added Iterator class, base class for all iterators in Freestyle (on the C++ side).

Maxime Curioni maxime.curioni at gmail.com
Wed Jul 23 12:19:09 CEST 2008


Revision: 15710
          http://projects.blender.org/plugins/scmsvn/viewcvs.php?view=rev&root=bf-blender&revision=15710
Author:   mxcurioni
Date:     2008-07-23 12:19:08 +0200 (Wed, 23 Jul 2008)

Log Message:
-----------
soc-2008-mxcurioni: added Iterator class, base class for all iterators in Freestyle (on the C++ side). Created the equivalent in Python BPy_Iterator with the simple interface:
- getExactTypeName()
- increment()
- decrement()
- isBegin()
- isEnd()

Contrary to previously stated, I am reverting back to implementing iterators in the (Python) API, for different reasons:
- it will make testing quicker to achieve, as I won't have to recode a big chunk of the original Python files
- it will be a base for API refactoring
- it won't prevent the use a list-based approach later (it is simple to get it from the Iterator)

Modified Paths:
--------------
    branches/soc-2008-mxcurioni/source/blender/freestyle/SConscript
    branches/soc-2008-mxcurioni/source/blender/freestyle/intern/python/BPy_Freestyle.cpp
    branches/soc-2008-mxcurioni/source/blender/freestyle/intern/stroke/ChainingIterators.h
    branches/soc-2008-mxcurioni/source/blender/freestyle/intern/swig/ModuleWrapper.cpp
    branches/soc-2008-mxcurioni/source/blender/freestyle/intern/swig/ModuleWrapper.h
    branches/soc-2008-mxcurioni/source/blender/freestyle/intern/view_map/Interface0D.h
    branches/soc-2008-mxcurioni/source/blender/freestyle/intern/view_map/ViewMapAdvancedIterators.h
    branches/soc-2008-mxcurioni/source/blender/freestyle/intern/view_map/ViewMapIterators.h
    branches/soc-2008-mxcurioni/source/blender/freestyle/python/Freestyle.py

Added Paths:
-----------
    branches/soc-2008-mxcurioni/source/blender/freestyle/intern/python/BPy_Iterator.cpp
    branches/soc-2008-mxcurioni/source/blender/freestyle/intern/python/BPy_Iterator.h
    branches/soc-2008-mxcurioni/source/blender/freestyle/intern/system/Iterator.cpp
    branches/soc-2008-mxcurioni/source/blender/freestyle/intern/system/Iterator.h

Modified: branches/soc-2008-mxcurioni/source/blender/freestyle/SConscript
===================================================================
--- branches/soc-2008-mxcurioni/source/blender/freestyle/SConscript	2008-07-23 07:56:08 UTC (rev 15709)
+++ branches/soc-2008-mxcurioni/source/blender/freestyle/SConscript	2008-07-23 10:19:08 UTC (rev 15710)
@@ -74,6 +74,7 @@
 					prefix + '/BPy_Interface1D.cpp',
 					prefix + '/Interface1D/BPy_FEdge.cpp',
 					prefix + '/Interface1D/BPy_Stroke.cpp',
+					prefix + '/BPy_Iterator.cpp',
 					prefix + '/BPy_MediumType.cpp',
 					prefix + '/BPy_Nature.cpp',
 					prefix + '/BPy_StrokeAttribute.cpp',

Modified: branches/soc-2008-mxcurioni/source/blender/freestyle/intern/python/BPy_Freestyle.cpp
===================================================================
--- branches/soc-2008-mxcurioni/source/blender/freestyle/intern/python/BPy_Freestyle.cpp	2008-07-23 07:56:08 UTC (rev 15709)
+++ branches/soc-2008-mxcurioni/source/blender/freestyle/intern/python/BPy_Freestyle.cpp	2008-07-23 10:19:08 UTC (rev 15710)
@@ -5,6 +5,7 @@
 #include "BPy_Id.h"
 #include "BPy_IntegrationType.h"
 #include "BPy_Interface0D.h"
+#include "BPy_Iterator.h"
 #include "Interface0D/BPy_CurvePoint.h"
 #include "BPy_Interface1D.h"
 #include "BPy_MediumType.h"
@@ -142,6 +143,7 @@
 	IntegrationType_Init( module );
 	Interface0D_Init( module );
 	Interface1D_Init( module );
+	Iterator_Init( module );
 	StrokeAttribute_Init( module );
 	StrokeShader_Init( module );
 	UnaryFunction0D_Init( module );

Added: branches/soc-2008-mxcurioni/source/blender/freestyle/intern/python/BPy_Iterator.cpp
===================================================================
--- branches/soc-2008-mxcurioni/source/blender/freestyle/intern/python/BPy_Iterator.cpp	                        (rev 0)
+++ branches/soc-2008-mxcurioni/source/blender/freestyle/intern/python/BPy_Iterator.cpp	2008-07-23 10:19:08 UTC (rev 15710)
@@ -0,0 +1,176 @@
+#include "BPy_Iterator.h"
+
+#include "BPy_Convert.h"
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+///////////////////////////////////////////////////////////////////////////////////////////
+
+/*---------------  Python API function prototypes for Iterator instance  -----------*/
+static void Iterator___dealloc__(BPy_Iterator *self);
+static PyObject * Iterator___repr__(BPy_Iterator* self);
+
+static PyObject * Iterator_getExactTypeName(BPy_Iterator* self);
+static PyObject * Iterator_increment(BPy_Iterator* self);
+static PyObject * Iterator_decrement(BPy_Iterator* self);
+static PyObject * Iterator_isBegin(BPy_Iterator* self);
+static PyObject * Iterator_isEnd(BPy_Iterator* self);
+
+/*----------------------Iterator instance definitions ----------------------------*/
+static PyMethodDef BPy_Iterator_methods[] = {
+	{"getExactTypeName", ( PyCFunction ) Iterator_getExactTypeName, METH_NOARGS, "( )Returns the string of the name of the iterator."},
+	{"increment", ( PyCFunction ) Iterator_increment, METH_NOARGS, "( )Increments iterator."},
+	{"decrement", ( PyCFunction ) Iterator_decrement, METH_NOARGS, "( )Decrements iterator."},
+	{"isBegin", ( PyCFunction ) Iterator_isBegin, METH_NOARGS, "( )Tests if iterator points to beginning."},
+	{"isEnd", ( PyCFunction ) Iterator_isEnd, METH_NOARGS, "( )Tests if iterator points to end."},
+	{NULL, NULL, 0, NULL}
+};
+
+/*-----------------------BPy_Iterator type definition ------------------------------*/
+
+PyTypeObject Iterator_Type = {
+	PyObject_HEAD_INIT( NULL ) 
+	0,							/* ob_size */
+	"Iterator",				/* tp_name */
+	sizeof( BPy_Iterator ),	/* tp_basicsize */
+	0,							/* tp_itemsize */
+	
+	/* methods */
+	(destructor)Iterator___dealloc__,	/* tp_dealloc */
+	NULL,                       				/* printfunc tp_print; */
+	NULL,                       				/* getattrfunc tp_getattr; */
+	NULL,                       				/* setattrfunc tp_setattr; */
+	NULL,										/* tp_compare */
+	(reprfunc)Iterator___repr__,					/* tp_repr */
+
+	/* Method suites for standard classes */
+
+	NULL,                       /* PyNumberMethods *tp_as_number; */
+	NULL,                       /* PySequenceMethods *tp_as_sequence; */
+	NULL,                       /* PyMappingMethods *tp_as_mapping; */
+
+	/* More standard operations (here for binary compatibility) */
+
+	NULL,						/* hashfunc tp_hash; */
+	NULL,                       /* ternaryfunc tp_call; */
+	NULL,                       /* reprfunc tp_str; */
+	NULL,                       /* getattrofunc tp_getattro; */
+	NULL,                       /* setattrofunc tp_setattro; */
+
+	/* Functions to access object as input/output buffer */
+	NULL,                       /* PyBufferProcs *tp_as_buffer; */
+
+  /*** Flags to define presence of optional/expanded features ***/
+	Py_TPFLAGS_DEFAULT | Py_TPFLAGS_BASETYPE, 		/* long tp_flags; */
+
+	NULL,                       /*  char *tp_doc;  Documentation string */
+  /*** Assigned meaning in release 2.0 ***/
+	/* call function for all accessible objects */
+	NULL,                       /* traverseproc tp_traverse; */
+
+	/* delete references to contained objects */
+	NULL,                       /* inquiry tp_clear; */
+
+  /***  Assigned meaning in release 2.1 ***/
+  /*** rich comparisons ***/
+	NULL,                       /* richcmpfunc tp_richcompare; */
+
+  /***  weak reference enabler ***/
+	0,                          /* long tp_weaklistoffset; */
+
+  /*** Added in release 2.2 ***/
+	/*   Iterators */
+	NULL,                       /* getiterfunc tp_iter; */
+	NULL,                       /* iternextfunc tp_iternext; */
+
+  /*** Attribute descriptor and subclassing stuff ***/
+	BPy_Iterator_methods,	/* struct PyMethodDef *tp_methods; */
+	NULL,                       	/* struct PyMemberDef *tp_members; */
+	NULL,         					/* struct PyGetSetDef *tp_getset; */
+	NULL,							/* struct _typeobject *tp_base; */
+	NULL,							/* PyObject *tp_dict; */
+	NULL,							/* descrgetfunc tp_descr_get; */
+	NULL,							/* descrsetfunc tp_descr_set; */
+	0,                          	/* long tp_dictoffset; */
+	NULL,                       	/* initproc tp_init; */
+	NULL,							/* allocfunc tp_alloc; */
+	PyType_GenericNew,		/* newfunc tp_new; */
+	
+	/*  Low-level free-memory routine */
+	NULL,                       /* freefunc tp_free;  */
+	
+	/* For PyObject_IS_GC */
+	NULL,                       /* inquiry tp_is_gc;  */
+	NULL,                       /* PyObject *tp_bases; */
+	
+	/* method resolution order */
+	NULL,                       /* PyObject *tp_mro;  */
+	NULL,                       /* PyObject *tp_cache; */
+	NULL,                       /* PyObject *tp_subclasses; */
+	NULL,                       /* PyObject *tp_weaklist; */
+	NULL
+};
+
+//-------------------MODULE INITIALIZATION--------------------------------
+PyMODINIT_FUNC Iterator_Init( PyObject *module )
+{
+	
+	if( module == NULL )
+		return;
+
+	if( PyType_Ready( &Iterator_Type ) < 0 )
+		return;
+	Py_INCREF( &Iterator_Type );
+	PyModule_AddObject(module, "Iterator", (PyObject *)&Iterator_Type);
+	
+}
+
+//------------------------INSTANCE METHODS ----------------------------------
+
+void Iterator___dealloc__(BPy_Iterator* self)
+{
+	delete self->it;
+    self->ob_type->tp_free((PyObject*)self);
+}
+
+PyObject * Iterator___repr__(BPy_Iterator* self)
+{
+    return PyString_FromFormat("type: %s - address: %p", self->it->getExactTypeName().c_str(), self->it );
+}
+
+PyObject * Iterator_getExactTypeName(BPy_Iterator* self) {
+	return PyString_FromString( self->it->getExactTypeName().c_str() );	
+}
+
+
+PyObject * Iterator_increment(BPy_Iterator* self) {
+	self->it->increment();
+		
+	Py_RETURN_NONE;
+}
+
+PyObject * Iterator_decrement(BPy_Iterator* self) {
+	self->it->decrement();
+		
+	Py_RETURN_NONE;
+}
+
+PyObject * Iterator_isBegin(BPy_Iterator* self) {
+	return PyBool_from_bool( self->it->isBegin() );
+}
+
+PyObject * Iterator_isEnd(BPy_Iterator* self) {
+	return PyBool_from_bool( self->it->isEnd() );
+}
+
+
+
+///////////////////////////////////////////////////////////////////////////////////////////
+
+#ifdef __cplusplus
+}
+#endif
+
+

Added: branches/soc-2008-mxcurioni/source/blender/freestyle/intern/python/BPy_Iterator.h
===================================================================
--- branches/soc-2008-mxcurioni/source/blender/freestyle/intern/python/BPy_Iterator.h	                        (rev 0)
+++ branches/soc-2008-mxcurioni/source/blender/freestyle/intern/python/BPy_Iterator.h	2008-07-23 10:19:08 UTC (rev 15710)
@@ -0,0 +1,35 @@
+#ifndef FREESTYLE_PYTHON_ITERATOR_H
+#define FREESTYLE_PYTHON_ITERATOR_H
+
+#include "../system/Iterator.h"
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+
+///////////////////////////////////////////////////////////////////////////////////////////
+
+#include <Python.h>
+
+extern PyTypeObject Iterator_Type;
+
+#define BPy_Iterator_Check(v)	(( (PyObject *) v)->ob_type == &Iterator_Type)
+
+/*---------------------------Python BPy_Iterator structure definition----------*/
+typedef struct {
+	PyObject_HEAD
+	Iterator *it;
+} BPy_Iterator;
+
+/*---------------------------Python BPy_Iterator visible prototypes-----------*/
+
+PyMODINIT_FUNC Iterator_Init( PyObject *module );
+
+
+///////////////////////////////////////////////////////////////////////////////////////////
+
+#ifdef __cplusplus
+}
+#endif
+
+#endif /* FREESTYLE_PYTHON_ITERATOR_H */

Modified: branches/soc-2008-mxcurioni/source/blender/freestyle/intern/stroke/ChainingIterators.h
===================================================================
--- branches/soc-2008-mxcurioni/source/blender/freestyle/intern/stroke/ChainingIterators.h	2008-07-23 07:56:08 UTC (rev 15709)
+++ branches/soc-2008-mxcurioni/source/blender/freestyle/intern/stroke/ChainingIterators.h	2008-07-23 10:19:08 UTC (rev 15710)
@@ -34,13 +34,15 @@
 # include "../view_map/ViewMapAdvancedIterators.h"
 # include "Predicates1D.h"
 
+#include "../system/Iterator.h" //soc 
+
 //using namespace ViewEdgeInternal;
 
 //
 // Adjacency iterator used in the chaining process
 //
 ///////////////////////////////////////////////////////////
-class LIB_STROKE_EXPORT AdjacencyIterator{
+class LIB_STROKE_EXPORT AdjacencyIterator : Iterator {
 protected:
   ViewVertexInternal::orientedViewEdgeIterator _internalIterator;
   bool _restrictToSelection;
@@ -71,10 +73,10 @@
   virtual ~AdjacencyIterator(){
   }
   
-  inline bool isEnd(){
+  virtual inline bool isEnd(){
     return _internalIterator.isEnd();
   }
-  inline bool isBegin(){
+  virtual inline bool isBegin(){

@@ Diff output truncated at 10240 characters. @@




More information about the Bf-blender-cvs mailing list