Skip to content

Commit

Permalink
fix(aws): align declaration and init order
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.

stack-info: PR: #557, branch: aws-nslick/stack/4
Signed-off-by: Nicholas Sielicki <nslick@amazon.com>
  • Loading branch information
Nicholas Sielicki committed Sep 5, 2024
1 parent ce2837b commit 1abc4e2
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 1abc4e2

Please sign in to comment.