Amebis/stdex
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

stream: weasel around false-positive GCC warning

Browse Source 
As _countof is manually implemented on non-Windows platforms, GCC
knows it returns size_t, but fails to realize it will never be more than
0x400 in our case making truncation harmless.

Signed-off-by: Simon Rozman <simon@rozman.si>
This commit is contained in:
Simon Rozman 2024-09-30 13:25:55 +02:00
parent 4fa30d5630
commit 9fb899849d
2 changed files with 4 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
include/stdex/stream.hpp
View File

@ -479,8 +479,9 @@ namespace stdex
return *this; return *this;
data.reserve(num_chars); data.reserve(num_chars);
for (;;) { for (;;) {
T buf[0x400]; constexpr uint32_t buf_chars = 0x400;
uint32_t num_read = static_cast<uint32_t>(read_array(buf, sizeof(T), std::min<uint32_t>(num_chars, _countof(buf)))); T buf[buf_chars];
uint32_t num_read = static_cast<uint32_t>(read_array(buf, sizeof(T), std::min<uint32_t>(num_chars, buf_chars)));
data.append(buf, num_read); data.append(buf, num_read);
num_chars -= num_read; num_chars -= num_read;
if (!num_chars || !ok()) if (!num_chars || !ok())

2
include/stdex/uuid.hpp
View File

@ -1,4 +1,4 @@
/* /*
SPDX-License-Identifier: MIT SPDX-License-Identifier: MIT
Copyright © 2016-2024 Amebis Copyright © 2016-2024 Amebis
*/ */