Lightemerald/windows-user-space-emulator
2
0
Fork 0
mirror of https://github.com/momo5502/emulator.git synced 2026-01-11 08:36:16 +00:00
Code Issues Packages Projects Releases Wiki Activity

Use stack-allocated ctx_t array instead of heap allocation

Browse Source 
- Initialize ctxs[i] directly with aggregate initialization
- Pass address of stack object to CreateThread
- Use dot notation instead of arrow for member access
- Remove delete calls since no heap allocation

Co-authored-by: Maurice Heumann <momo5502@gmail.com>
This commit is contained in:
Elias Bachaalany
2025-12-02 15:53:37 -08:00
parent 4a905d562a
commit 651c020b87
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/samples/test-sample/test.cpp
View File

@@ -111,8 +111,8 @@ namespace
for (int i = 0; i < thread_count; i++)
{
ctx_t* ctx = ctxs[i] = new ctx_t{5 * (i + 1), i - 2};
threads[i] = CreateThread(nullptr, 0, thread_proc, ctx, 0, nullptr);
ctxs[i] = {5 * (i + 1), 0};
threads[i] = CreateThread(nullptr, 0, thread_proc, &ctxs[i], 0, nullptr);
if (!threads[i])
{
return false;
@@ -124,12 +124,11 @@ namespace
const int expected_results[thread_count] = {5, 10, 15, 20, 25};
for (int i = 0; i < thread_count; i++)
{
if (ctxs[i]->result != expected_results[i])
if (ctxs[i].result != expected_results[i])
{
return false;
}
CloseHandle(threads[i]);
delete ctxs[i];
}
return true;