Skip to content

Commit 4c2b56b

Browse files
committed
fix: review issues
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
1 parent a201d77 commit 4c2b56b

3 files changed

Lines changed: 4 additions & 3 deletions

File tree

include/scale/detail/compact_integer_jam.hpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,8 @@ namespace scale {
7878

7979
// NOLINTNEXTLINE(*-pro-type-member-init)
8080
std::array<uint8_t, sizeof(size_t) + 1> bytes;
81-
uint8_t &prefix = bytes[0] = 0;
81+
uint8_t &prefix = bytes[0];
82+
prefix = 0;
8283
size_t len = 1;
8384

8485
for (decltype(value) i = value; i != 0; i >>= 8) {

include/scale/detail/fixed_width_integer.hpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,7 @@ namespace scale {
104104
* @param encoder SCALE encoder.
105105
*/
106106
inline void encode(bool boolean, Encoder &encoder) {
107-
encoder.put(boolean ? 1u : 0u);
107+
encoder.put(boolean ? 1 : 0);
108108
}
109109

110110
/**

test/scale_tune_test.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ struct Object : private qtils::Empty {
7575
}
7676

7777
friend void decode(Object &obj, Decoder &decoder) {
78-
auto mul = decoder. getConfig<MulConfig>().multi;
78+
auto mul = decoder.getConfig<MulConfig>().multi;
7979
auto add = decoder.getConfig<AddConfig>().add;
8080
size_t size;
8181
decode(as_compact(size), decoder);

0 commit comments

Comments
 (0)