fix for blend_render_info.py not closing file & some edits to comments.
authorCampbell Barton <ideasman42@gmail.com>
Wed, 22 Aug 2012 10:29:30 +0000 (10:29 +0000)
committerCampbell Barton <ideasman42@gmail.com>
Wed, 22 Aug 2012 10:29:30 +0000 (10:29 +0000)
release/scripts/modules/blend_render_info.py
release/scripts/templates/operator_mesh_add.py
source/blender/makesdna/DNA_curve_types.h
source/blender/makesdna/DNA_sequence_types.h

index 8762ea0e287392e80fe5266af52fa4bc2051e4b0..d08b9e09ddaa4e4da094795875143ef7a1456d3a 100755 (executable)
@@ -86,6 +86,8 @@ def read_blend_rend_chunk(path):
 
         scenes.append((start_frame, end_frame, scene_name))
 
+    blendfile.close()
+
     return scenes
 
 
index f2bb231cb7a8a7e08d54585ab54b8e56d62fe297..fa248cb9005c812798c3d08de021070def390689 100644 (file)
@@ -78,9 +78,9 @@ class AddBox(bpy.types.Operator):
     def execute(self, context):
 
         verts_loc, faces = add_box(self.width,
-                                     self.height,
-                                     self.depth,
-                                     )
+                                   self.height,
+                                   self.depth,
+                                   )
 
         mesh = bpy.data.meshes.new("Box")
 
index 56cf90d3d9d09a547ac0521f997e06453fde73e8..284694f2b48224081d0e426f6f00ad4d556b7662 100644 (file)
@@ -87,20 +87,22 @@ typedef struct BevPoint {
        short split_tag, dupe_tag;
 } BevPoint;
 
-/* Keyframes on F-Curves (allows code reuse of Bezier eval code) and 
+/**
+ * Keyframes on F-Curves (allows code reuse of Bezier eval code) and
  * Points on Bezier Curves/Paths are generally BezTriples 
- */
-/* note: alfa location in struct is abused by Key system */
-/* vec in BezTriple looks like this:
- * - vec[0][0]=x location of handle 1
- * - vec[0][1]=y location of handle 1
- * - vec[0][2]=z location of handle 1 (not used for FCurve Points(2d))
- * - vec[1][0]=x location of control point
- * - vec[1][1]=y location of control point
- * - vec[1][2]=z location of control point
- * - vec[2][0]=x location of handle 2
- * - vec[2][1]=y location of handle 2
- * - vec[2][2]=z location of handle 2 (not used for FCurve Points(2d))
+ *
+ * \note alfa location in struct is abused by Key system
+ *
+ * \note vec in BezTriple looks like this:
+ * - vec[0][0] = x location of handle 1
+ * - vec[0][1] = y location of handle 1
+ * - vec[0][2] = z location of handle 1 (not used for FCurve Points(2d))
+ * - vec[1][0] = x location of control point
+ * - vec[1][1] = y location of control point
+ * - vec[1][2] = z location of control point
+ * - vec[2][0] = x location of handle 2
+ * - vec[2][1] = y location of handle 2
+ * - vec[2][2] = z location of handle 2 (not used for FCurve Points(2d))
  */
 typedef struct BezTriple {
        float vec[3][3];
@@ -119,6 +121,10 @@ typedef struct BPoint {
        float radius, pad;              /* user-set radius per point for beveling etc */
 } BPoint;
 
+/**
+ * \note Nurb name is misleading, since it can be used for polygons too,
+ * also, it should be NURBS (Nurb isn't the singular of Nurbs).
+ */
 typedef struct Nurb {
        struct Nurb *next, *prev;       /* multiple nurbs per curve object are allowed */
        short type;
index e7ab6f6640893ec0f06aed6040cdc4c280cd9268..0835bb6b4c8f7c11c5241c16db94c37abbdda7b8 100644 (file)
@@ -102,11 +102,16 @@ typedef struct Strip {
        StripColorBalance *color_balance;
 } Strip;
 
-/* The sequence structure is the basic struct used by any strip. each of the strips uses a different sequence structure.*/
-/* WATCH IT: first part identical to ID (for use in ipo's)
- * the commend above is historic, probably we can drop the ID compatibility, but take care making this change */
-
-/* WATCH ITv2, this is really a 'Strip' in the UI!, name is highly confusing */
+/**
+ * The sequence structure is the basic struct used by any strip.
+ * each of the strips uses a different sequence structure.
+ *
+ * \warning The first part identical to ID (for use in ipo's)
+ * the commend above is historic, probably we can drop the ID compatibility,
+ * but take care making this change.
+ *
+ * \warning This is really a 'Strip' in the UI!, name is highly confusing.
+ */
 typedef struct Sequence {
        struct Sequence *next, *prev;
        void *tmp; /* tmp var for copying, and tagging for linked selection */