Skip to content

Commit

Permalink
fix(aws): align declaration and init order (#557)
Browse files Browse the repository at this point in the history
When using designated initializer lists, under C++ only, a warning will
be emitted if the declaration order does not match the initialization
order if any fields are missing in the initializer list. Reorder these
structs to initialize all fields in their declaration order.
Signed-off-by: Nicholas Sielicki <nslick@amazon.com>
  • Loading branch information
aws-nslick authored Sep 10, 2024
1 parent fdebc77 commit ce214aa
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions src/platform-aws.c
Original file line number Diff line number Diff line change
Expand Up @@ -95,16 +95,16 @@ struct ec2_platform_data {
},
{
.name = "trn1.32xlarge",
.default_protocol = "RDMA",
.gdr_required = true,
.net_flush_required = true,
.default_protocol = "RDMA",
.domain_per_thread = 1,
},
{
.name = "trn1n.32xlarge",
.default_protocol = "RDMA",
.gdr_required = true,
.net_flush_required = true,
.default_protocol = "RDMA",
.domain_per_thread = 1,
}
};
Expand Down

0 comments on commit ce214aa

Please sign in to comment.