Potential fix for #41

README.md

@@ -1,7 +1,7 @@
 quaggaJS
 ========
 
-- [Changelog](#changelog) (2015-06-09)
+- [Changelog](#changelog) (2015-06-13)
 
 ## What is QuaggaJS?
 
@@ -299,6 +299,10 @@ work.
 
 ## <a name="changelog">Changelog</a>
 
+### 2015-06-13
+- Improvements
+  - Added fixes for ``Code39Reader`` (trailing whitespace was missing)
+
 ### 2015-06-09
 - Features
   - Introduced the ``area`` property

dist/quagga.js

@@ -6890,7 +6890,13 @@ define(
             } while(decodedChar !== '*');
             result.pop();
 
+            if (!result.length) {
+                return null;
+            }
 
+            if(!self._verifyTrailingWhitespace(lastStart, nextStart, counters)) {
+                return null;
+            }
 
             return {
                 code : result.join(""),
@@ -6901,6 +6907,17 @@ define(
             };
         };
 
+        Code39Reader.prototype._verifyTrailingWhitespace = function(lastStart, nextStart, counters) {
+            var trailingWhitespaceEnd,
+                patternSize = ArrayHelper.sum(counters);
+
+            trailingWhitespaceEnd = nextStart - lastStart - patternSize;
+            if ((trailingWhitespaceEnd * 3) >= patternSize) {
+                return true;
+            }
+            return false;
+        };
+
         Code39Reader.prototype._patternToChar = function(pattern) {
             var i,
                 self = this;
@@ -7902,12 +7919,12 @@ define('frame_grabber',["cv_utils"], function(CVUtils) {
         _canvas.height = _canvasSize.y;
         _ctx = _canvas.getContext("2d");
         _data = new Uint8Array(_size.x * _size.y);
-        console.log("FrameGrabber", {
+        console.log("FrameGrabber", JSON.stringify({
             size: _size,
             topRight: topRight,
             videoSize: _video_size,
             canvasSize: _canvasSize
-        });
+        }));
 
         /**
          * Uses the given array as frame-buffer 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "quagga",
-  "version": "0.6.7",
+  "version": "0.6.8",
   "description": "An advanced barcode-scanner written in JavaScript",
   "main": "dist/quagga.js",
   "devDependencies": {

src/code_39_reader.js

@@ -82,7 +82,13 @@ define(
             } while(decodedChar !== '*');
             result.pop();
 
+            if (!result.length) {
+                return null;
+            }
 
+            if(!self._verifyTrailingWhitespace(lastStart, nextStart, counters)) {
+                return null;
+            }
 
             return {
                 code : result.join(""),
@@ -93,6 +99,17 @@ define(
             };
         };
 
+        Code39Reader.prototype._verifyTrailingWhitespace = function(lastStart, nextStart, counters) {
+            var trailingWhitespaceEnd,
+                patternSize = ArrayHelper.sum(counters);
+
+            trailingWhitespaceEnd = nextStart - lastStart - patternSize;
+            if ((trailingWhitespaceEnd * 3) >= patternSize) {
+                return true;
+            }
+            return false;
+        };
+
         Code39Reader.prototype._patternToChar = function(pattern) {
             var i,
                 self = this;

src/frame_grabber.js

@@ -24,12 +24,12 @@ define(["cv_utils"], function(CVUtils) {
         _canvas.height = _canvasSize.y;
         _ctx = _canvas.getContext("2d");
         _data = new Uint8Array(_size.x * _size.y);
-        console.log("FrameGrabber", {
+        console.log("FrameGrabber", JSON.stringify({
             size: _size,
             topRight: topRight,
             videoSize: _video_size,
             canvasSize: _canvasSize
-        });
+        }));
 
         /**
          * Uses the given array as frame-buffer