Skip to content

Commit 7534d48

Browse files
JakeStevensJacob Robert Stevens
andauthored
Fix unsigned comparison to remove compiler warning on embedded builds (#12446)
### Summary Internal builds for embedded targets treat type limit warnings as errors. There is an unsigned comparison against zero, which generates an error. This simple PR removes it. ### Test plan CI. Tested in successful internal build Co-authored-by: Jacob Robert Stevens <[email protected]>
1 parent abd6eff commit 7534d48

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

runtime/executor/merged_data_map.h

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ class MergedDataMap final : public NamedDataMap {
114114
ET_NODISCARD Result<const char*> get_key(uint32_t index) const override {
115115
uint32_t total_num_keys = get_num_keys().get();
116116
ET_CHECK_OR_RETURN_ERROR(
117-
index >= 0 && index < total_num_keys,
117+
index < total_num_keys,
118118
InvalidArgument,
119119
"Index %" PRIu32 " out of range of size %" PRIu32,
120120
index,

0 commit comments

Comments
 (0)