Style cleanup: C & pep8
authorCampbell Barton <ideasman42@gmail.com>
Tue, 15 Apr 2014 03:11:48 +0000 (13:11 +1000)
committerCampbell Barton <ideasman42@gmail.com>
Tue, 15 Apr 2014 03:11:48 +0000 (13:11 +1000)
doc/python_api/rst_from_bmesh_opdefines.py
intern/cycles/blender/addon/ui.py
release/scripts/modules/bpy/ops.py
release/scripts/templates_py/custom_nodes.py
release/scripts/templates_py/operator_modal_view3d_raycast.py
release/scripts/templates_py/ui_panel.py
source/blender/editors/sculpt_paint/paint_stroke.c
source/blender/editors/space_view3d/drawobject.c
source/blender/makesrna/intern/rna_mesh.c
source/tests/bl_run_operators.py
source/tests/pep8.py

index 977d3171393cc3a6a964d2fb1737e0c72fec7199..32afe335089ec1d44dd8f9e05ee12a126edd677f 100644 (file)
@@ -113,7 +113,7 @@ def main():
     fsrc.close()
     del fsrc
 
-   # namespace hack
+    # namespace hack
     vars = (
         "BMO_OP_SLOT_ELEMENT_BUF",
         "BMO_OP_SLOT_BOOL",
index e7c5e58a16f42669bc0d9853ca15f2dd7ae4811e..a064b4ff4c9074281a14af9bdd11b1495615fc8a 100644 (file)
@@ -47,12 +47,12 @@ class CyclesButtonsPanel():
     def poll(cls, context):
         rd = context.scene.render
         return rd.engine in cls.COMPAT_ENGINES
-        
-        
+
+
 def use_cpu(context):
     cscene = context.scene.cycles
     device_type = context.user_preferences.system.compute_device_type
-    
+
     return (device_type == 'NONE' or cscene.device == 'CPU')
 
 
@@ -1126,7 +1126,7 @@ class CyclesTexture_PT_colors(CyclesButtonsPanel, Panel):
     def poll(cls, context):
         # node = context.texture_node
         return False
-        #return node and CyclesButtonsPanel.poll(context)
+        # return node and CyclesButtonsPanel.poll(context)
 
     def draw(self, context):
         layout = self.layout
index e457c087ab82f277fffc6ffefa982b01b9271ba3..2be2abc638391b054f73a50e58ce118439746d1b 100644 (file)
@@ -204,8 +204,10 @@ class BPyOpsSubModOp(object):
         import bpy
         idname = self.idname()
         as_string = op_as_string(idname)
-        # XXX You never quite know what you get from bpy.types, with operators... Operator and OperatorProperties
-        #     are shadowing each other, and not in the same way for native ops and py ones! See T39158.
+        # XXX You never quite know what you get from bpy.types,
+        # with operators... Operator and OperatorProperties
+        # are shadowing each other, and not in the same way for
+        # native ops and py ones! See T39158.
         # op_class = getattr(bpy.types, idname)
         op_class = op_get_rna(idname)
         descr = op_class.bl_rna.description
index bf89c0debad4dca95ecbd98c549123d33667535a..992ef734859e600ec2f8bb0ae38e7c2baa84ce09 100644 (file)
@@ -49,13 +49,15 @@ class MyCustomSocket(NodeSocket):
     def draw_color(self, context, node):
         return (1.0, 0.4, 0.216, 0.5)
 
+
 # Mix-in class for all custom nodes in this tree type.
 # Defines a poll function to enable instantiation.
-class MyCustomTreeNode :
+class MyCustomTreeNode:
     @classmethod
     def poll(cls, ntree):
         return ntree.bl_idname == 'CustomTreeType'
 
+
 # Derived from the Node base type.
 class MyCustomNode(Node, MyCustomTreeNode):
     # === Basics ===
@@ -123,6 +125,7 @@ class MyCustomNode(Node, MyCustomTreeNode):
 import nodeitems_utils
 from nodeitems_utils import NodeCategory, NodeItem
 
+
 # our own base class with an appropriate poll function,
 # so the categories only show up in our own tree type
 class MyNodeCategory(NodeCategory):
@@ -143,12 +146,12 @@ node_categories = [
         # NB: settings values are stored as string expressions,
         # for this reason they should be converted to strings using repr()
         NodeItem("CustomNodeType", label="Node A", settings={
-            "myStringProperty" : repr("Lorem ipsum dolor sit amet"),
-            "myFloatProperty" : repr(1.0),
+            "myStringProperty": repr("Lorem ipsum dolor sit amet"),
+            "myFloatProperty": repr(1.0),
             }),
         NodeItem("CustomNodeType", label="Node B", settings={
-            "myStringProperty" : repr("consectetur adipisicing elit"),
-            "myFloatProperty" : repr(2.0),
+            "myStringProperty": repr("consectetur adipisicing elit"),
+            "myFloatProperty": repr(2.0),
             }),
         ]),
     ]
index eac76922187b9cee6cd13f021d0ed0f169bcd0e4..4d0355047f499f90e6959d5ee4f630963fab29ca 100644 (file)
@@ -16,7 +16,6 @@ def main(context, event, ray_max=10000.0):
     ray_origin = view3d_utils.region_2d_to_origin_3d(region, rv3d, coord)
     ray_target = ray_origin + (view_vector * ray_max)
 
-
     def visible_objects_and_duplis():
         """Loop over (object, matrix) pairs (mesh only)"""
 
index cacdb83e81553d33b5737a99ca71f848ede3c1e5..855d05700c56a8f9a2480303647513288da29b91 100644 (file)
@@ -42,22 +42,22 @@ class LayoutDemoPanel(bpy.types.Panel):
         col.label(text="Column Two:")
         col.prop(scene, "frame_start")
         col.prop(scene, "frame_end")
-        
+
         # Big render button
         layout.label(text="Big Button:")
         row = layout.row()
         row.scale_y = 3.0
         row.operator("render.render")
-        
+
         # Different sizes in a row
         layout.label(text="Different button sizes:")
         row = layout.row(align=True)
         row.operator("render.render")
-        
+
         sub = row.row()
         sub.scale_x = 2.0
         sub.operator("render.render")
-        
+
         row.operator("render.render")
 
 
index d3b6d8cdd1a384a6cf77f355cd18d0a8c2cf4b17..7b9121446f562e1cfbce407e503c422278d51b15 100644 (file)
@@ -169,7 +169,7 @@ static void paint_brush_update(bContext *C, Brush *brush, PaintMode mode,
         *      brush coord/pressure/etc.
         *      It's more an events design issue, which doesn't split coordinate/pressure/angle
         *      changing events. We should avoid this after events system re-design */
-       if(!stroke->brush_init) {
+       if (!stroke->brush_init) {
                copy_v2_v2(stroke->initial_mouse, mouse);
                copy_v2_v2(ups->last_rake, mouse);
                copy_v2_v2(ups->tex_mouse, mouse);
index 24e00dd49e21415fe733ba30383a70342f8f53a2..df34f1f6f6517337239f96cd65b6374820b17912 100644 (file)
@@ -2414,8 +2414,8 @@ static void draw_dm_loop_normals(BMEditMesh *em, Scene *scene, Object *ob, Deriv
 
        if (lnors) {
                drawDMNormal_userData data;
-               MLoop *mloops = dm->getLoopArray(dm);
-               MVert *mverts = dm->getVertArray(dm);
+               const MLoop *mloops = dm->getLoopArray(dm);
+               const MVert *mverts = dm->getVertArray(dm);
                int i, totloops = dm->getNumLoops(dm);
 
                data.bm = em->bm;
@@ -2426,7 +2426,7 @@ static void draw_dm_loop_normals(BMEditMesh *em, Scene *scene, Object *ob, Deriv
                glBegin(GL_LINES);
                for (i = 0; i < totloops; i++, mloops++, lnors++) {
                        float no[3];
-                       float *co = mverts[mloops->v].co;
+                       const float *co = mverts[mloops->v].co;
 
                        if (!data.uniform_scale) {
                                mul_v3_m3v3(no, data.tmat, (float *)lnors);
@@ -2434,7 +2434,7 @@ static void draw_dm_loop_normals(BMEditMesh *em, Scene *scene, Object *ob, Deriv
                                mul_m3_v3(data.imat, no);
                        }
                        else {
-                               copy_v3_v3(no,(float *)lnors);
+                               copy_v3_v3(no, (float *)lnors);
                        }
                        mul_v3_fl(no, data.normalsize);
                        add_v3_v3(no, co);
index 99c35bdcff28b71aac7d3a590878715ba3eedb71..42f316ee20882749d92477a8ff7d75bcfd888c27 100644 (file)
@@ -424,7 +424,7 @@ static void rna_MeshTessFace_split_normals_get(PointerRNA *ptr, float *values)
                while (i--) zero_v3(&values[i * 3]);
        }
        else {
-               while (i--) normal_short_to_float_v3(&values[i * 3], (const short*)(*vec)[i]);
+               while (i--) normal_short_to_float_v3(&values[i * 3], (const short *)(*vec)[i]);
        }
 }
 
index 13bf615b8dcd986bd165428bfa86b6c7613b2eb3..e14b0ce6d32eba65f3044d173d7c54d31814e4ea 100644 (file)
@@ -105,7 +105,7 @@ def blend_list(mainpath):
 
     def is_blend(filename):
         ext = splitext(filename)[1]
-        return (ext in {".blend",})
+        return (ext in {".blend", })
 
     return list(sorted(file_list(mainpath, is_blend)))
 
index 3ae1828c59a33c279a39c661df7c81229185233c..235bca48104fab19527210b5c47384465a8b93f0 100644 (file)
@@ -54,7 +54,7 @@ def is_pep8(path):
         print("\nfile contains BOM, remove first 3 bytes: %r\n" % path)
 
     # templates don't have a header but should be pep8
-    for d in ("presets", "templates", "examples"):
+    for d in ("presets", "templates_py", "examples"):
         if ("%s%s%s" % (os.sep, d, os.sep)) in path:
             return 1