Re: [PATCH] Added debug filtering to avoid debugger to start when using in production

[Oliver Gutierrez <ogutierrez@xxxxxxxxxx>]

This change:

-**  display, inputs, and cursor channels.  See main.js for
+**  display, inputs, and cursor channels.  See main.js for

Is because file in repository has a trailing whitespace.

It happened before in another patch I sent but now is when I finally found the reason.

On Mon, Sep 5, 2016 at 10:48 AM, Oliver Gutierrez <ogutierrez@xxxxxxxxxx> wrote:

---
 spiceconn.js | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/spiceconn.js b/spiceconn.js
index 903f3b0..5123e04 100644
--- a/spiceconn.js
+++ b/spiceconn.js
@@ -23,7 +23,7 @@
 **      This is the base _javascript_ class for establishing and
 **  managing a connection to a Spice Server.
 **  It is used to provide core functionality to the Spice main,
-**  display, inputs, and cursor channels.  See main.js for
+**  display, inputs, and cursor channels.  See main.js for
 **  usage.
 **--------------------------------------------------------------------------*/
 function SpiceConn(o)
@@ -185,8 +185,8 @@ SpiceConn.prototype =

                 if (msg.type > 500)
                 {
-                    alert("Something has gone very wrong; we think we have message of type " + msg.type);
-                    debugger;
+                    DEBUG > 0 && alert("Something has gone very wrong; we think we have message of type " + msg.type);
+                    DEBUG > 0 && debugger;
                 }

                 if (msg.size == 0)
--
2.7.4

--

Oliver Gutierrez
Associate Software Engineer - Desktop Management tools
Red Hat

_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/spice-devel