[Bf-extensions-cvs] SVN commit: /data/svn/bf-extensions [2623] contrib/py/scripts/addons/ add_mesh_pyramid.py: minor edits, use extend() in a more efficient way.

Campbell Barton ideasman42 at gmail.com
Thu Nov 17 22:46:36 CET 2011


Revision: 2623
          http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-extensions&revision=2623
Author:   campbellbarton
Date:     2011-11-17 21:46:36 +0000 (Thu, 17 Nov 2011)
Log Message:
-----------
minor edits, use extend() in a more efficient way.

Modified Paths:
--------------
    contrib/py/scripts/addons/add_mesh_pyramid.py

Modified: contrib/py/scripts/addons/add_mesh_pyramid.py
===================================================================
--- contrib/py/scripts/addons/add_mesh_pyramid.py	2011-11-17 21:32:59 UTC (rev 2622)
+++ contrib/py/scripts/addons/add_mesh_pyramid.py	2011-11-17 21:46:36 UTC (rev 2623)
@@ -1,5 +1,3 @@
-# add_mesh_pyramid.py (c) 2011 Phil Cote (cotejrp1)
-#
 # ***** BEGIN GPL LICENSE BLOCK *****
 #
 #
@@ -18,6 +16,9 @@
 # Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
 #
 # ***** END GPL LICENCE BLOCK *****
+
+# (c) 2011 Phil Cote (cotejrp1)
+
 bl_info = {
     'name': 'Mesh Pyramid',
     'author': 'Phil Cote, cotejrp1, (http://www.blenderaddons.com)',
@@ -31,7 +32,10 @@
 
 
 import bpy
-from bpy.props import FloatVectorProperty, IntProperty, FloatProperty, BoolProperty
+from bpy.props import (FloatVectorProperty,
+                       IntProperty,
+                       FloatProperty,
+                       BoolProperty)
 
 from add_utils import AddObjectHelper, add_object_data
 from mathutils import Vector
@@ -66,25 +70,21 @@
         vert_list.extend((bfl, bfr, bbl, bbr, tfl, tfr, tbl, tbr,))
 
         # side faces
-        face_list.extend(((voffset + 4, voffset + 5,
-                            voffset + 1, voffset + 0),))  # front
-        face_list.extend(((voffset + 6, voffset + 7,
-                            voffset + 3, voffset + 2),))  # back
-        face_list.extend(((voffset + 2, voffset + 6,
-                            voffset + 4, voffset + 0),))  # left
-        face_list.extend(((voffset + 3, voffset + 7,
-                            voffset + 5, voffset + 1),))  # right
+        face_list.extend((
+            (voffset + 4, voffset + 5, voffset + 1, voffset + 0), # back
+            (voffset + 6, voffset + 7, voffset + 3, voffset + 2), # front
+            (voffset + 2, voffset + 6, voffset + 4, voffset + 0), # left
+            (voffset + 3, voffset + 7, voffset + 5, voffset + 1), # right
+            ))
 
         # horizontal connecting faces ( note: n/a for the first iteration ).
         if voffset > 0:
-            face_list.extend(((voffset - 4, voffset - 3,
-                            voffset + 1, voffset + 0),))  # connector front
-            face_list.extend(((voffset - 2, voffset - 1,
-                            voffset + 3, voffset + 2),))  # back
-            face_list.extend(((voffset - 4, voffset - 2,
-                            voffset + 2, voffset + 0),))  # left
-            face_list.extend(((voffset - 3, voffset - 1,
-                            voffset + 3, voffset + 1),))  # right
+            face_list.extend((
+                (voffset - 4, voffset - 3, voffset + 1, voffset + 0), # connector front
+                (voffset - 2, voffset - 1, voffset + 3, voffset + 2), # back
+                (voffset - 4, voffset - 2, voffset + 2, voffset + 0), # left
+                (voffset - 3, voffset - 1, voffset + 3, voffset + 1), # right
+                ))
 
         # set up parameters for the next iteration
         cur_size = cur_size - (step_width * 2)
@@ -94,13 +94,12 @@
         sn = sn + 1
         voffset = voffset + 8
 
-    # cap the top.
     voffset = voffset - 8  # remove extra voffset done on final iteration
-    face_list.extend(((voffset + 6, voffset + 7, voffset + 5, voffset + 4),))
+    face_list.extend((
+        (voffset + 6, voffset + 7, voffset + 5, voffset + 4), # cap the top.
+        (2, 3, 1, 0), # cap the bottom.
+        ))
 
-    # cap the bottom.
-    face_list.extend(((2, 3, 1, 0),))
-
     return vert_list, face_list
 
 



More information about the Bf-extensions-cvs mailing list