<p><a href="https://review.coreboot.org/18902">View Change</a></p><p>8 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/18902/13/util/intelvbttool/intelvbttool.c">File util/intelvbttool/intelvbttool.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/18902/13/util/intelvbttool/intelvbttool.c@421">Patch Set #13, Line 421:</a> <code style="font-family:monospace,monospace">    struct fileobject* fo = malloc_fo(size);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo* bar" should be "foo *bar"</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/18902/13/util/intelvbttool/intelvbttool.c@773">Patch Set #13, Line 773:</a> <code style="font-family:monospace,monospace">            (const struct vbt_header *)fo->data;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">code indent should use tabs where possible</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/18902/13/util/intelvbttool/intelvbttool.c@808">Patch Set #13, Line 808:</a> <code style="font-family:monospace,monospace">      /* Duplicate fo as caller is owner of fo and remalloc frees the object */</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/18902/13/util/intelvbttool/intelvbttool.c@845">Patch Set #13, Line 845:</a> <code style="font-family:monospace,monospace">       if (oh->size == 0 || oh->size > 0x80|| oh->size * 512 > fo->size) {</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/18902/13/util/intelvbttool/intelvbttool.c@855">Patch Set #13, Line 855:</a> <code style="font-family:monospace,monospace">                } else {</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">else is not generally useful after a break or return</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/18902/13/util/intelvbttool/intelvbttool.c@867">Patch Set #13, Line 867:</a> <code style="font-family:monospace,monospace">           pcir = (const optionrom_pcir_t *)((const u8 *)oh + oh->pcir_offset);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/18902/13/util/intelvbttool/intelvbttool.c@924">Patch Set #13, Line 924:</a> <code style="font-family:monospace,monospace">         const struct fileobject* fo_vbt = malloc_fo_sub(fo, i);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">"foo* bar" should be "foo *bar"</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/18902/13/util/intelvbttool/intelvbttool.c@964">Patch Set #13, Line 964:</a> <code style="font-family:monospace,monospace">     bdb = (const struct bdb_header *)((const char *)head + head->bdb_offset);</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">line over 80 characters</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/18902">change 18902</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/18902"/><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: I8cbde042c7f5632f36648419becd23e248ba6f76 </div>
<div style="display:none"> Gerrit-Change-Number: 18902 </div>
<div style="display:none"> Gerrit-PatchSet: 13 </div>
<div style="display:none"> Gerrit-Owner: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: Martin Roth <martinroth@google.com> </div>
<div style="display:none"> Gerrit-Reviewer: Nico Huber <nico.h@gmx.de> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <patrick.rudolph@9elements.com> </div>
<div style="display:none"> Gerrit-Reviewer: Patrick Rudolph <siro@das-labor.org> </div>
<div style="display:none"> Gerrit-Reviewer: Paul Menzel <paulepanter@users.sourceforge.net> </div>
<div style="display:none"> Gerrit-Reviewer: Samuel Holland <samuel@sholland.org> </div>
<div style="display:none"> Gerrit-Reviewer: Stefan Reinauer <stefan.reinauer@coreboot.org> </div>
<div style="display:none"> Gerrit-Reviewer: Vladimir Serbinenko <phcoder@gmail.com> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Wed, 06 Jun 2018 16:35:52 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>