[aseprite] 205/250: Spray tool now works with small widths and speeds. (fix #838)

[Tobias Hansen]

This is an automated email from the git hooks/post-receive script.

thansen pushed a commit to branch master
in repository aseprite.

commit 5c451d2cfdf4ccc5837493a66bfe0c79a631ea37
Author: Aravin <aravin at mentis.ca>
Date:   Tue Oct 27 19:18:59 2015 -0400

    Spray tool now works with small widths and speeds. (fix #838)
---
 src/app/tools/point_shapes.h | 39 +++++++++++++++++++++++----------------
 1 file changed, 23 insertions(+), 16 deletions(-)

diff --git a/src/app/tools/point_shapes.h b/src/app/tools/point_shapes.h
index 77e4a01..a3fb490 100644
--- a/src/app/tools/point_shapes.h
+++ b/src/app/tools/point_shapes.h
@@ -131,6 +131,7 @@ private:
 
 class SprayPointShape : public PointShape {
   BrushPointShape m_subPointShape;
+  float m_pointRemainder = 0;
 
 public:
 
@@ -143,31 +144,37 @@ public:
   void transformPoint(ToolLoop* loop, int x, int y) override {
     int spray_width = loop->getSprayWidth();
     int spray_speed = loop->getSpraySpeed();
-    int c, u, v, times = (spray_width*spray_width/4) * spray_speed / 100;
 
-    // In Windows, rand() has a RAND_MAX too small
-#if RAND_MAX <= 0xffff
-    fixmath::fixed angle, radius;
+    // The number of points to spray is proportional to the spraying area, and
+    // we calculate it as a float to handle very low spray rates properly.
+    float points_to_spray = (spray_width * spray_width / 4.0f) * spray_speed / 100.0f;
 
-    for (c=0; c<times; c++) {
-      angle = fixmath::itofix(rand() * 256 / RAND_MAX);
-      radius = fixmath::itofix(rand() * (spray_width*10) / RAND_MAX) / 10;
-      u = fixmath::fixtoi(fixmath::fixmul(radius, fixmath::fixcos(angle)));
-      v = fixmath::fixtoi(fixmath::fixmul(radius, fixmath::fixsin(angle)));
+    // We add the fractional points from last time to get
+    // the total number of points to paint this time.
+    points_to_spray += m_pointRemainder;
+    int integral_points = (int)points_to_spray;
+
+    // Save any leftover fraction of a point for next time.
+    m_pointRemainder = points_to_spray - integral_points;
+    ASSERT(m_pointRemainder >= 0 && m_pointRemainder < 1.0f);
 
-      m_subPointShape.transformPoint(loop, x+u, y+v);
-    }
-#else
     fixmath::fixed angle, radius;
 
-    for (c=0; c<times; c++) {
+    for (int c=0; c<integral_points; c++) {
+
+#if RAND_MAX <= 0xffff
+      // In Windows, rand() has a RAND_MAX too small
+      angle = fixmath::itofix(rand() * 255 / RAND_MAX);
+      radius = fixmath::itofix(rand() * spray_width / RAND_MAX);
+#else
       angle = rand();
       radius = rand() % fixmath::itofix(spray_width);
-      u = fixmath::fixtoi(fixmath::fixmul(radius, fixmath::fixcos(angle)));
-      v = fixmath::fixtoi(fixmath::fixmul(radius, fixmath::fixsin(angle)));
+#endif
+
+      int u = fixmath::fixtoi(fixmath::fixmul(radius, fixmath::fixcos(angle)));
+      int v = fixmath::fixtoi(fixmath::fixmul(radius, fixmath::fixsin(angle)));
       m_subPointShape.transformPoint(loop, x+u, y+v);
     }
-#endif
   }
 
   void getModifiedArea(ToolLoop* loop, int x, int y, Rect& area) override {

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-games/aseprite.git