<p><a href="https://review.coreboot.org/27015">View Change</a></p><p>9 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/27015/3/util/msrtool/linux.c">File util/msrtool/linux.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27015/3/util/msrtool/linux.c@71">Patch Set #3, Line 71:</a> <code style="font-family:monospace,monospace">     return ret ==0;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">spaces required around that '==' (ctx:WxV)</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/27015/3/util/msrtool/msrtool.c">File util/msrtool/msrtool.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27015/3/util/msrtool/msrtool.c@128">Patch Set #3, Line 128:</a> <code style="font-family:monospace,monospace">                     if ((fin = fopen(streamfn, "r")) == NULL) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">do not use assignment in if condition</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27015/3/util/msrtool/msrtool.c@136">Patch Set #3, Line 136:</a> <code style="font-family:monospace,monospace">                 if ((fdout = mkstemp(tmpfn)) == -1) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">do not use assignment in if condition</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27015/3/util/msrtool/msrtool.c@140">Patch Set #3, Line 140:</a> <code style="font-family:monospace,monospace">                 if ((fout = fdopen(fdout, "w")) == NULL) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">do not use assignment in if condition</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27015/3/util/msrtool/msrtool.c@145">Patch Set #3, Line 145:</a> <code style="font-family:monospace,monospace">                  if ((fout = fopen(streamfn, "w")) == NULL) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">do not use assignment in if condition</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27015/3/util/msrtool/msrtool.c@216">Patch Set #3, Line 216:</a> <code style="font-family:monospace,monospace">        else if ((fin = fopen(difffn, "r")) == NULL) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">do not use assignment in if condition</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27015/3/util/msrtool/msrtool.c@226">Patch Set #3, Line 226:</a> <code style="font-family:monospace,monospace">              for (len = strlen(m1start) - 1; strchr("\r\n", m1start[len]) != NULL; --len)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/27015/3/util/msrtool/msrutils.c">File util/msrtool/msrutils.c:</a></p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27015/3/util/msrtool/msrutils.c@325">Patch Set #3, Line 325:</a> <code style="font-family:monospace,monospace">      if (m= NULL) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">spaces required around that '=' (ctx:VxW)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/27015/3/util/msrtool/msrutils.c@325">Patch Set #3, Line 325:</a> <code style="font-family:monospace,monospace">   if (m= NULL) {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">do not use assignment in if condition</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/27015">change 27015</a>. To unsubscribe, or for help writing mail filters, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/27015"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: comment </div>
<div style="display:none"> Gerrit-Change-Id: I76d35a3643600f81a6da7e0af99c935ebd1c2fc7 </div>
<div style="display:none"> Gerrit-Change-Number: 27015 </div>
<div style="display:none"> Gerrit-PatchSet: 3 </div>
<div style="display:none"> Gerrit-Owner: Elyes HAOUAS <ehaouas@noos.fr> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-CC: Patrick Georgi <pgeorgi@google.com> </div>
<div style="display:none"> Gerrit-Comment-Date: Mon, 11 Jun 2018 09:54:15 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>