Skip to content

Commit a097eaa

Browse files
committed
Initialize fields to suppress gcc warnings.
1 parent c1dd29e commit a097eaa

File tree

2 files changed

+17
-8
lines changed

2 files changed

+17
-8
lines changed

demo/ci_create_discovery/ci_create_discovery.cpp

+10-7
Original file line numberDiff line numberDiff line change
@@ -26,13 +26,16 @@ int main() {
2626
constexpr std::size_t buffer_size = 256;
2727
constexpr midi2::ci::header header{
2828
.device_id = 0_b7, .version = 2_b7, .remote_muid = my_muid, .local_muid = midi2::ci::broadcast_muid};
29-
constexpr midi2::ci::discovery_reply discovery{.manufacturer = {0x12_b7, 0x23_b7, 0x34_b7},
30-
.family = 0x1779_b14,
31-
.model = 0x2B5D_b14,
32-
.version = {0x01_b7, 0x00_b7, 0x00_b7, 0x00_b7},
33-
.capability = 0x7F_b7,
34-
.max_sysex_size = midi2::ci::b28{buffer_size},
35-
.output_path_id = 0_b7};
29+
constexpr midi2::ci::discovery_reply discovery{
30+
.manufacturer = {0x12_b7, 0x23_b7, 0x34_b7},
31+
.family = 0x1779_b14,
32+
.model = 0x2B5D_b14,
33+
.version = {0x01_b7, 0x00_b7, 0x00_b7, 0x00_b7},
34+
.capability = 0x7F_b7,
35+
.max_sysex_size = midi2::ci::b28{buffer_size},
36+
.output_path_id = 0_b7,
37+
.function_block = 0_b7,
38+
};
3639
std::array<std::byte, buffer_size> message{};
3740
// NOLINTNEXTLINE(llvm-qualified-auto,readability-qualified-auto)
3841
auto const first = std::begin(message);

include/midi2/ci_types.hpp

+7-1
Original file line numberDiff line numberDiff line change
@@ -1225,7 +1225,11 @@ struct capabilities {
12251225
};
12261226

12271227
constexpr capabilities capabilities::make(packed::capabilities_v1 const &other) noexcept {
1228-
return {.num_simultaneous = b7{to_underlying(other.num_simultaneous)}};
1228+
return {
1229+
.num_simultaneous = b7{to_underlying(other.num_simultaneous)},
1230+
.major_version = b7{},
1231+
.minor_version = b7{},
1232+
};
12291233
}
12301234
constexpr capabilities capabilities::make(packed::capabilities_v2 const &other) noexcept {
12311235
return {
@@ -1289,6 +1293,8 @@ struct capabilities_reply {
12891293
constexpr capabilities_reply capabilities_reply::make(packed::capabilities_reply_v1 const &other) noexcept {
12901294
return {
12911295
.num_simultaneous = b7{to_underlying(other.num_simultaneous)},
1296+
.major_version = b7{},
1297+
.minor_version = b7{},
12921298
};
12931299
}
12941300
constexpr capabilities_reply capabilities_reply::make(packed::capabilities_reply_v2 const &other) noexcept {

0 commit comments

Comments
 (0)