<div dir="ltr">Ok. Thanks for the tips :D<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Sep 8, 2016 at 2:26 PM, Pavel Grunt <span dir="ltr"><<a href="mailto:pgrunt@redhat.com" target="_blank">pgrunt@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ah, I didn't realized before pushing /o\<br>
<br>
Please make the summary shorter - you can put the other sentence to<br>
the commit body.<br>
<span class=""><br>
On Thu, 2016-09-08 at 13:21 +0200, Oliver Gutierrez wrote:<br>
> ---<br>
>  spiceconn.js | 6 ++++--<br>
>  1 file changed, 4 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/spiceconn.js b/spiceconn.js<br>
> index 010a446..796f954 100644<br>
> --- a/spiceconn.js<br>
> +++ b/spiceconn.js<br>
> @@ -189,8 +189,10 @@ SpiceConn.prototype =<br>
>  <br>
>                  if (msg.type > 500)<br>
>                  {<br>
> -                    DEBUG > 0 && alert("Something has gone very<br>
> wrong; we think we have message of type " + msg.type);<br>
> -                    DEBUG > 0 && debugger;<br>
> +                    if (DEBUG > 0) {<br>
</span>Please align to our brackets & indentation style <br>
<span class="">> +                      alert("<wbr>Something has gone very wrong; we<br>
> think we have message of type " + msg.type);<br>
> +                      <wbr>debugger;<br>
> +                    }<br>
>                  }<br>
>  <br>
>                  if (msg.size == 0)<br>
</span>Besides that it works & looks good to me<br>
<br>
Thanks,<br>
Pavel<br>
</blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><span>Oliver Gutierrez<br>Associate Software Engineer - Desktop Management tools<br>Red Hat</span></div></div>
</div>