1 files changed, 23 insertions, 19 deletions

diff --git a/src/library_sdl.js b/src/library_sdl.js
index 124e71f6..9d947cd4 100644
--- a/src/library_sdl.js
+++ b/src/library_sdl.js
@@ -173,17 +173,21 @@ mergeInto(LibraryManager.library, {
     },
 
     // Load SDL color into a CSS-style color specification
-    loadColorToCSS: function(color) {
+    loadColorToCSSRGB: function(color) {
       var rgba = {{{ makeGetValue('color', '0', 'i32') }}};
-      return 'rgba(' + (rgba&255) + ',' + ((rgba >> 8)&255) + ',' + ((rgba >> 16)&255) + ',' + (1-((rgba >> 24)&255)/255) + ')';
+      return 'rgb(' + (rgba&255) + ',' + ((rgba >> 8)&255) + ',' + ((rgba >> 16)&255) + ')';
+    },
+    loadColorToCSSRGBA: function(color) {
+      var rgba = {{{ makeGetValue('color', '0', 'i32') }}};
+      return 'rgba(' + (rgba&255) + ',' + ((rgba >> 8)&255) + ',' + ((rgba >> 16)&255) + ',' + (((rgba >> 24)&255)/255) + ')';
     },
 
-    translateColorToCSS: function(rgba) {
-      return 'rgba(' + ((rgba >> 24)&255) + ',' + ((rgba >> 16)&255) + ',' + ((rgba >> 8)&255) + ',' + (1-(rgba&255)/255) + ')';
+    translateColorToCSSRGBA: function(rgba) {
+      return 'rgba(' + ((rgba >> 24)&255) + ',' + ((rgba >> 16)&255) + ',' + ((rgba >> 8)&255) + ',' + ((rgba&255)/255) + ')';
     },
 
-    translateRGBAToCSS: function(r, g, b, a) {
-      return 'rgba(' + r + ',' + g + ',' + b + ',' + (1-a/255) + ')';
+    translateRGBAToCSSRGBA: function(r, g, b, a) {
+      return 'rgba(' + r + ',' + g + ',' + b + ',' + (a/255) + ')';
     },
 
     makeSurface: function(width, height, flags, usePageCanvas, source) {
@@ -345,7 +349,7 @@ mergeInto(LibraryManager.library, {
       var tempCtx = SDL.surfaces[SDL.screen].ctx;
       tempCtx.save();
       tempCtx.font = fontString;
-      var ret = tempCtx.measureText(text).width;
+      var ret = tempCtx.measureText(text).width | 0;
       tempCtx.restore();
       return ret;
     },
@@ -579,14 +583,14 @@ mergeInto(LibraryManager.library, {
     assert(!surfData.locked); // but we could unlock and re-lock if we must..
     var r = SDL.loadRect(rect);
     surfData.ctx.save();
-    surfData.ctx.fillStyle = SDL.translateColorToCSS(color);
+    surfData.ctx.fillStyle = SDL.translateColorToCSSRGBA(color);
     surfData.ctx.fillRect(r.x, r.y, r.w, r.h);
     surfData.ctx.restore();
   },
 
   SDL_BlitSurface__deps: ['SDL_UpperBlit'],
   SDL_BlitSurface: function(src, srcrect, dst, dstrect) {
-    return _SDL_Blit(src, srcrect, dst, dstrect);
+    return _SDL_UpperBlit(src, srcrect, dst, dstrect);
   },
 
   SDL_SetAlpha: function(surf, flag, alpha) {
@@ -737,15 +741,15 @@ mergeInto(LibraryManager.library, {
     return id;
   },
 
-  Mix_FreeChunk: function(audio) {
-    SDL.audios[id].audio.pause();
-    SDL.audios[id] = null;
+  Mix_FreeChunk: function(id) {
+    //SDL.audios[id].audio.pause();
+    //SDL.audios[id] = null;
     return 0;
   },
 
-  Mix_PlayChannel: function(channel, audio, loops) {
-    var audio = SDL.audios[id].audio;
-    audio.play();
+  Mix_PlayChannel: function(channel, id, loops) {
+    //var audio = SDL.audios[id].audio;
+    //audio.play();
     return 0; // XXX should return channel
   },
   Mix_PlayChannelTimed: 'Mix_PlayChannel', // XXX ignore Timing
@@ -815,7 +819,7 @@ mergeInto(LibraryManager.library, {
     var fontData = SDL.fonts[font];
     var w = SDL.estimateTextWidth(fontData, text);
     var h = fontData.size;
-    var color = SDL.loadColorToCSS(color);
+    var color = SDL.loadColorToCSSRGB(color); // XXX alpha breaks fonts?
     var fontString = h + 'px sans-serif';
     var surf = SDL.makeSurface(w, h, 0, false, 'text:' + text); // bogus numbers..
     var surfData = SDL.surfaces[surf];
@@ -858,7 +862,7 @@ mergeInto(LibraryManager.library, {
     assert(!surfData.locked); // but we could unlock and re-lock if we must..
     // TODO: if ctx does not change, leave as is, and also do not re-set xStyle etc.
     surfData.ctx.save();
-    surfData.ctx.fillStyle = SDL.translateRGBAToCSS(r, g, b, a);
+    surfData.ctx.fillStyle = SDL.translateRGBAToCSSRGBA(r, g, b, a);
     surfData.ctx.fillRect(x1, y1, x2-x1, y2-y1);
     surfData.ctx.restore();
   },
@@ -867,7 +871,7 @@ mergeInto(LibraryManager.library, {
     var surfData = SDL.surfaces[surf];
     assert(!surfData.locked); // but we could unlock and re-lock if we must..
     surfData.ctx.save();
-    surfData.ctx.strokeStyle = SDL.translateRGBAToCSS(r, g, b, a);
+    surfData.ctx.strokeStyle = SDL.translateRGBAToCSSRGBA(r, g, b, a);
     surfData.ctx.strokeRect(x1, y1, x2-x1, y2-y1);
     surfData.ctx.restore();
   },
@@ -876,7 +880,7 @@ mergeInto(LibraryManager.library, {
     var surfData = SDL.surfaces[surf];
     assert(!surfData.locked); // but we could unlock and re-lock if we must..
     surfData.ctx.save();
-    surfData.ctx.strokeStyle = SDL.translateRGBAToCSS(r, g, b, a);
+    surfData.ctx.strokeStyle = SDL.translateRGBAToCSSRGBA(r, g, b, a);
     surfData.ctx.beginPath();
     surfData.ctx.moveTo(x1, y1);
     surfData.ctx.lineTo(x2, y2);