[Bf-blender-cvs] SVN commit: /data/svn/bf-blender [56873] trunk/blender/source/blender/ editors/sculpt_paint/paint_ops.c: Fix #35393: Sculpt Shortcuts are bananas.

[Sergey Sharybin]

Revision: 56873
          
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=56873
Author:   nazgul
Date:     2013-05-17 11:08:50 +0000 (Fri, 17 May 2013)
Log Message:
-----------
Fix #35393: Sculpt Shortcuts are bananas.

Issue was introduced by self in svn rev56815.
Brushes list was simply not fully traversing
due to mistake in termination condition.

Revision Links:
--------------
    
http://projects.blender.org/scm/viewvc.php?view=rev&root=bf-blender&revision=56815

Modified Paths:
--------------
    trunk/blender/source/blender/editors/sculpt_paint/paint_ops.c

Modified: trunk/blender/source/blender/editors/sculpt_paint/paint_ops.c
===================================================================
--- trunk/blender/source/blender/editors/sculpt_paint/paint_ops.c       
2013-05-17 09:47:05 UTC (rev 56872)
+++ trunk/blender/source/blender/editors/sculpt_paint/paint_ops.c       
2013-05-17 11:08:50 UTC (rev 56873)
@@ -271,17 +271,17 @@
        }
 
        /* get the next brush with the active tool */
-       for (brush = first_brush;
-            brush != brush_orig;
-            brush = brush->id.next ? brush->id.next : bmain->brush.first)
-       {
+       brush = first_brush;
+       do {
                if ((brush->ob_mode & ob_mode) &&
                    (brush_tool(brush, tool_offset) == tool))
                {
                        return brush;
                }
-       }
 
+               brush = brush->id.next ? brush->id.next : bmain->brush.first;
+       } while (brush != first_brush);
+
        return NULL;
 }
 

_______________________________________________
Bf-blender-cvs mailing list
Bf-blender-cvs@blender.org
http://lists.blender.org/mailman/listinfo/bf-blender-cvs