<p><a href="https://review.coreboot.org/28208">View Change</a></p><p>34 comments:</p><ul style="list-style: none; padding: 0;"><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lz4/lib/xxhash.c">File util/cbfstool/lz4/lib/xxhash.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/28208/3/util/cbfstool/lz4/lib/xxhash.c@68">Patch Set #3, Line 68:</a> <code style="font-family:monospace,monospace"> * By default, xxHash library provides endian-independent Hash values, based on little-endian convention.</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@222">Patch Set #3, Line 222:</a> <code style="font-family:monospace,monospace">typedef enum { XXH_bigEndian=0, XXH_littleEndian=1 } XXH_endianness;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">spaces required around that '=' (ctx:VxV)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lz4/lib/xxhash.c@222">Patch Set #3, Line 222:</a> <code style="font-family:monospace,monospace">typedef enum { XXH_bigEndian=0, XXH_littleEndian=1 } XXH_endianness;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">spaces required around that '=' (ctx:VxV)</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lz4/lib/xxhash.c@236">Patch Set #3, Line 236:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U32 XXH_readLE32_align(const void* ptr, XXH_endianness endian, XXH_alignment align)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@236">Patch Set #3, Line 236:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U32 XXH_readLE32_align(const void* ptr, XXH_endianness endian, XXH_alignment align)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@244">Patch Set #3, Line 244:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U32 XXH_readLE32(const void* ptr, XXH_endianness endian)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@249">Patch Set #3, Line 249:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U64 XXH_readLE64_align(const void* ptr, XXH_endianness endian, XXH_alignment align)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@249">Patch Set #3, Line 249:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U64 XXH_readLE64_align(const void* ptr, XXH_endianness endian, XXH_alignment align)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@257">Patch Set #3, Line 257:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U64 XXH_readLE64(const void* ptr, XXH_endianness endian)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@288">Patch Set #3, Line 288:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U32 XXH32_endian_align(const void* input, size_t len, U32 seed, XXH_endianness endian, XXH_alignment align)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@288">Patch Set #3, Line 288:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U32 XXH32_endian_align(const void* input, size_t len, U32 seed, XXH_endianness endian, XXH_alignment align)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@374">Patch Set #3, Line 374:</a> <code style="font-family:monospace,monospace">    XXH_endianness endian_detected = (XXH_endianness)XXH_CPU_LITTLE_ENDIAN;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lz4/lib/xxhash.c@393">Patch Set #3, Line 393:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U64 XXH64_endian_align(const void* input, size_t len, U64 seed, XXH_endianness endian, XXH_alignment align)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@393">Patch Set #3, Line 393:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U64 XXH64_endian_align(const void* input, size_t len, U64 seed, XXH_endianness endian, XXH_alignment align)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@514">Patch Set #3, Line 514:</a> <code style="font-family:monospace,monospace">    XXH_endianness endian_detected = (XXH_endianness)XXH_CPU_LITTLE_ENDIAN;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lz4/lib/xxhash.c@615">Patch Set #3, Line 615:</a> <code style="font-family:monospace,monospace">FORCE_INLINE XXH_errorcode XXH32_update_endian (XXH32_state_t* state_in, const void* input, size_t len, XXH_endianness endian)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@615">Patch Set #3, Line 615:</a> <code style="font-family:monospace,monospace">FORCE_INLINE XXH_errorcode XXH32_update_endian (XXH32_state_t* state_in, const void* input, size_t len, XXH_endianness endian)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@615">Patch Set #3, Line 615:</a> <code style="font-family:monospace,monospace">FORCE_INLINE XXH_errorcode XXH32_update_endian (XXH32_state_t* state_in, const void* input, size_t len, XXH_endianness endian)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@615">Patch Set #3, Line 615:</a> <code style="font-family:monospace,monospace">FORCE_INLINE XXH_errorcode XXH32_update_endian (XXH32_state_t* state_in, const void* input, size_t len, XXH_endianness endian)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space prohibited between function name and open parenthesis '('</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lz4/lib/xxhash.c@706">Patch Set #3, Line 706:</a> <code style="font-family:monospace,monospace">    XXH_endianness endian_detected = (XXH_endianness)XXH_CPU_LITTLE_ENDIAN;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lz4/lib/xxhash.c@716">Patch Set #3, Line 716:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U32 XXH32_digest_endian (const XXH32_state_t* state_in, XXH_endianness endian)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@716">Patch Set #3, Line 716:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U32 XXH32_digest_endian (const XXH32_state_t* state_in, XXH_endianness endian)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@716">Patch Set #3, Line 716:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U32 XXH32_digest_endian (const XXH32_state_t* state_in, XXH_endianness endian)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space prohibited between function name and open parenthesis '('</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lz4/lib/xxhash.c@760">Patch Set #3, Line 760:</a> <code style="font-family:monospace,monospace">    XXH_endianness endian_detected = (XXH_endianness)XXH_CPU_LITTLE_ENDIAN;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lz4/lib/xxhash.c@769">Patch Set #3, Line 769:</a> <code style="font-family:monospace,monospace">FORCE_INLINE XXH_errorcode XXH64_update_endian (XXH64_state_t* state_in, const void* input, size_t len, XXH_endianness endian)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@769">Patch Set #3, Line 769:</a> <code style="font-family:monospace,monospace">FORCE_INLINE XXH_errorcode XXH64_update_endian (XXH64_state_t* state_in, const void* input, size_t len, XXH_endianness endian)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@769">Patch Set #3, Line 769:</a> <code style="font-family:monospace,monospace">FORCE_INLINE XXH_errorcode XXH64_update_endian (XXH64_state_t* state_in, const void* input, size_t len, XXH_endianness endian)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@769">Patch Set #3, Line 769:</a> <code style="font-family:monospace,monospace">FORCE_INLINE XXH_errorcode XXH64_update_endian (XXH64_state_t* state_in, const void* input, size_t len, XXH_endianness endian)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space prohibited between function name and open parenthesis '('</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lz4/lib/xxhash.c@860">Patch Set #3, Line 860:</a> <code style="font-family:monospace,monospace">    XXH_endianness endian_detected = (XXH_endianness)XXH_CPU_LITTLE_ENDIAN;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lz4/lib/xxhash.c@870">Patch Set #3, Line 870:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U64 XXH64_digest_endian (const XXH64_state_t* state_in, XXH_endianness endian)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@870">Patch Set #3, Line 870:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U64 XXH64_digest_endian (const XXH64_state_t* state_in, XXH_endianness endian)</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/28208/3/util/cbfstool/lz4/lib/xxhash.c@870">Patch Set #3, Line 870:</a> <code style="font-family:monospace,monospace">FORCE_INLINE U64 XXH64_digest_endian (const XXH64_state_t* state_in, XXH_endianness endian)</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">space prohibited between function name and open parenthesis '('</p></li><li style="margin: 0; padding: 0 0 0 16px;"><p style="margin-bottom: 4px;"><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lz4/lib/xxhash.c@954">Patch Set #3, Line 954:</a> <code style="font-family:monospace,monospace">    XXH_endianness endian_detected = (XXH_endianness)XXH_CPU_LITTLE_ENDIAN;</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li></ul></li><li style="margin: 0; padding: 0;"><p><a href="https://review.coreboot.org/#/c/28208/3/util/cbfstool/lzma/C/LzmaEnc.h">File util/cbfstool/lzma/C/LzmaEnc.h:</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/28208/3/util/cbfstool/lzma/C/LzmaEnc.h@40">Patch Set #3, Line 40:</a> <code style="font-family:monospace,monospace">  SZ_ERROR_PARAM  - Incorrect parameter in props</code></p><p style="white-space: pre-wrap; word-wrap: break-word;">please, no spaces at the start of a line</p></li></ul></li></ul><p>To view, visit <a href="https://review.coreboot.org/28208">change 28208</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/28208"/><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: I6967a106ce1286d633ddeeb041f582e65f9ea78c </div>
<div style="display:none"> Gerrit-Change-Number: 28208 </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: Paul Menzel <paulepanter@users.sourceforge.net> </div>
<div style="display:none"> Gerrit-Reviewer: build bot (Jenkins) <no-reply@coreboot.org> </div>
<div style="display:none"> Gerrit-Comment-Date: Thu, 23 Aug 2018 16:26:30 +0000 </div>
<div style="display:none"> Gerrit-HasComments: Yes </div>
<div style="display:none"> Gerrit-HasLabels: No </div>