Changeset 3350

Timestamp:

12/20/07 16:43:26 (13 months ago)

Author:

gyuque

Message:

lang/legacy-actionscript/as2draw: tuned line rendering algorithm... comparing with native API.

Location:

lang/legacy-actionscript/as2draw

Files:

• AS2Draw.swf (modified) (previous)
• Drawable.as (modified) (4 diffs)
• PixelBuffer.as (modified) (7 diffs)

lang/legacy-actionscript/as2draw/Drawable.as

@@ -14 +14 @@
                 mDragData = {dragging: false, prevX: 0, prevY: 0};
                 mMC = aContainer.createEmptyMovieClip("drawable_mc", 1);
+                mMC._quality = "LOW";
                 mPixBuf = new PixelBuffer(aW, aH, mMC);
                 mBGColor = aBGColor;
@@ -25 +26 @@
                 mMC.onMouseMove = function(){ _this.onMouseMove(); };
                 mMC.useHandCursor = false;
+                
+                // test patterns
+                renderLine(80, 20, 190, 20);
+                renderLine(80, 20, 130, 190);
+                renderLine(80, 20, 100, 190);
+                renderLine(80, 20, 80, 190);
+                renderLine(80, 20, 60, 190);
+                renderLine(80, 20, 10, 20);
+                
         }
 
@@ -64 +74 @@
                 var dx:Number = 0;
                 var dy:Number = 0;
-                
-                mMC.lineStyle(undefined, 0, 0, true);
+                l = 0;
                 for (var i:Number = -1;i < l;i++)
                 {
@@ -73 +82 @@
                                 dy = ofs[(i<<1)+1];
                         }
-                        mMC.beginFill(mCurrentPen.color);
                         mPixBuf.drawLine(
                                 x1 +dx, y1 +dy,
-                                x2 +dx, y2 +dy);
-                        mMC.endFill();
+                                x2 +dx, y2 +dy, 0, mCurrentPen.color);
+
                 }
+                
+                        mMC.lineStyle(1, 0xff0f00);
+                        mMC.moveTo(x1+dx, y1+dy);
+                        mMC.lineTo(x2+dx, y2+dy);
         }
 

lang/legacy-actionscript/as2draw/PixelBuffer.as

@@ -42 +42 @@
         private var mDir;
         private var mCount;
+        
         public function begin(aX:Number, aY:Number, aDir:Number)
         {
@@ -55 +56 @@
         }
 
-        public function commit(aData:Number)
+        public function commit(aData:Number, aOutColor:Number)
         {
+                if (mCount == 0)
+                        return;
+
                 var dx:Number;
                 var dy:Number;
 
+                mOut.lineStyle(1, aOutColor);
+                
                 switch(mDir) {
                 case 0:
                         dx = 0; dy = -1;
-                        U.rect(mOut, mSX, mSY-mCount, 1, mCount);
+                        mOut.moveTo(mSX, mSY-mCount);
+                        mOut.lineTo(mSX, mSY);
                         break;
                 case 1:
                         dx = 1; dy = 0;
-                        U.rect(mOut, mSX, mSY, mCount, 1);
+                        mOut.moveTo(mSX       , mSY);
+                        mOut.lineTo(mSX+mCount, mSY);
                         break;
                 case 2:
                         dx = 0; dy = 1;
-                        U.rect(mOut, mSX, mSY, 1, mCount);
+                        mOut.moveTo(mSX, mSY);
+                        mOut.lineTo(mSX, mSY+mCount);
                         break;
                 case 3:
                         dx = -1; dy = 0;
-                        U.rect(mOut, mSX-mCount, mSY, mCount, 1);
+                        mOut.moveTo(mSX-mCount, mSY);
+                        mOut.lineTo(mSX       , mSY);
                         break;
                 }
+                //mOut.moveTo(0,0);
 
                 for (var i:Number = 0;i < mCount;i++)
@@ -89 +100 @@
 
 
-        public function drawLine(x1:Number, y1:Number, x2:Number, y2:Number, c:Number)
+        public function drawLine(x1:Number, y1:Number, x2:Number, y2:Number, c:Number, outc:Number)
         {
                 x1 = Math.floor(x1);
@@ -100 +111 @@
                 var dy:Number = Math.abs(y2 - y1);
                 
-                E = 0;
-                
                 var x:Number;
                 var y:Number;
@@ -109 +118 @@
                 if(dx > dy)
                 {
+                        E = dx/2;
                         if (x2 < x1)
                         {
@@ -127 +137 @@
                                         E -= dx;
                                         y += step;
-                                        commit(c);
+                                        commit(c, outc);
                                         begin(x+1, y, 1);
                                 }                                       
                         }
-                        commit(c);
+                        commit(c, outc);
                 }
                 else
                 {
+                        E = dy/2;
                         if (y2 < y1)
                         {
@@ -155 +166 @@
                                         E -= dy;
                                         x += step;
-                                        commit(c);
+                                        commit(c, outc);
                                         begin(x, y+1, 2);
                                 }
                         }
-                        commit(c);
+                        commit(c, outc);
                 }
         }
 }
+