From 40db8c04fe35e3326645cf85edd2cf07ca19fd14 Mon Sep 17 00:00:00 2001 From: Parth Aggarwal Date: Thu, 2 Apr 2026 21:11:30 -0700 Subject: [PATCH 1/2] Update deps: c-build-tools, c-pal Dependency updates: c-build-tools: - 1a00465 Fix: guard .gitmodules access for repos without submodules (#376) - 473d3d1 [MrBot] Remove validation_msbuild_target to enable automatic submodule propagation (#375) - ada7a79 Fix orphaned else in update-c-build-tools-yaml-refs (#367) - b92332c [MrBot] Extract repo validation and traceability to run once instead of per-flavor (#369) c-pal: - a88947a [MrBot] Remove THANDLE from job_object_helper, return int instead (#562) --- build/devops_gated.yml | 2 +- deps/c-build-tools | 2 +- deps/c-logging | 2 +- deps/c-pal | 2 +- deps/c-testrunnerswitcher | 2 +- deps/c-util | 2 +- deps/ctest | 2 +- deps/macro-utils-c | 2 +- deps/umock-c | 2 +- 9 files changed, 9 insertions(+), 9 deletions(-) diff --git a/build/devops_gated.yml b/build/devops_gated.yml index 41e60e0..0614850 100644 --- a/build/devops_gated.yml +++ b/build/devops_gated.yml @@ -7,7 +7,7 @@ resources: type: github name: azure/c-build-tools endpoint: github.com_azure - ref: 61a603bed806edd8a3dc2f335a67314129fe59e2 + ref: 1a00465f8984d3e642f1f1616a5680b065c6f15e jobs: - template: /pipeline_templates/build_all_flavors.yml@c_build_tools parameters: diff --git a/deps/c-build-tools b/deps/c-build-tools index 61a603b..1a00465 160000 --- a/deps/c-build-tools +++ b/deps/c-build-tools @@ -1 +1 @@ -Subproject commit 61a603bed806edd8a3dc2f335a67314129fe59e2 +Subproject commit 1a00465f8984d3e642f1f1616a5680b065c6f15e diff --git a/deps/c-logging b/deps/c-logging index 0a3a397..e10d482 160000 --- a/deps/c-logging +++ b/deps/c-logging @@ -1 +1 @@ -Subproject commit 0a3a397c4e23cea7cfd5ec623a8779d4d2865b68 +Subproject commit e10d482f578a449438ba05242e5b9f2c27582a68 diff --git a/deps/c-pal b/deps/c-pal index f9d09c1..926096a 160000 --- a/deps/c-pal +++ b/deps/c-pal @@ -1 +1 @@ -Subproject commit f9d09c1dfbed8dac35f7375739d2db9a855d50e7 +Subproject commit 926096ae52fa32fcf9dd48aa101c84f5429b9762 diff --git a/deps/c-testrunnerswitcher b/deps/c-testrunnerswitcher index d2b5f7c..3b7cf4c 160000 --- a/deps/c-testrunnerswitcher +++ b/deps/c-testrunnerswitcher @@ -1 +1 @@ -Subproject commit d2b5f7c079f33db7e485bcc1dbd8d4b512eaf529 +Subproject commit 3b7cf4c9ced732e9e81916ed5470651a1d9f02b3 diff --git a/deps/c-util b/deps/c-util index 42dff72..b29d311 160000 --- a/deps/c-util +++ b/deps/c-util @@ -1 +1 @@ -Subproject commit 42dff728a35934ec88414476f3de055204b91118 +Subproject commit b29d311c454b29428956aca8b8a50815a4dec7f5 diff --git a/deps/ctest b/deps/ctest index f2c4f16..7c42b0f 160000 --- a/deps/ctest +++ b/deps/ctest @@ -1 +1 @@ -Subproject commit f2c4f167e48f385392466d6525e09b83025f27b9 +Subproject commit 7c42b0f4c3892fd363792ddd6c7b588a32976e82 diff --git a/deps/macro-utils-c b/deps/macro-utils-c index d409706..a1610e9 160000 --- a/deps/macro-utils-c +++ b/deps/macro-utils-c @@ -1 +1 @@ -Subproject commit d409706453c170966e6660a90287e3e0728edc77 +Subproject commit a1610e95f90a8a4cdbfd46ae7d37d1e23d5debec diff --git a/deps/umock-c b/deps/umock-c index 3801adc..9de4b33 160000 --- a/deps/umock-c +++ b/deps/umock-c @@ -1 +1 @@ -Subproject commit 3801adce25c97c47174a55223d03e1cb37fb44d8 +Subproject commit 9de4b33c8433c0f560954265300f6c1e1173c3e0 From ff4212bb5b6b19b9909af973afd055f32c8f87e6 Mon Sep 17 00:00:00 2001 From: Parth Aggarwal Date: Thu, 2 Apr 2026 23:47:02 -0700 Subject: [PATCH 2/2] Fix job_object_helper API usage after THANDLE removal in c-pal Update clds_hash_table_with_memory_limit_int.c to use the new non-THANDLE job_object_helper API: return type changed from THANDLE(JOB_OBJECT_HELPER) to int, and cleanup uses job_object_helper_deinit_for_test() instead of THANDLE_ASSIGN. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> --- .../clds_hash_table_with_memory_limit_int.c | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/tests/clds_hash_table_with_memory_limit_int/clds_hash_table_with_memory_limit_int.c b/tests/clds_hash_table_with_memory_limit_int/clds_hash_table_with_memory_limit_int.c index 5dbdb5b..c0837e3 100644 --- a/tests/clds_hash_table_with_memory_limit_int/clds_hash_table_with_memory_limit_int.c +++ b/tests/clds_hash_table_with_memory_limit_int/clds_hash_table_with_memory_limit_int.c @@ -17,7 +17,6 @@ #include "c_pal/gballoc_hl.h" #include "c_pal/gballoc_hl_redirect.h" -#include "c_pal/thandle.h" #include "c_pal/job_object_helper.h" #include "clds/clds_hazard_pointers.h" @@ -131,8 +130,8 @@ TEST_FUNCTION(clds_hash_table_insert_same_block_again_does_not_leak_item_count) } ASSERT_IS_TRUE(percent_to_use < 100, "Asking to use percent_to_use=%" PRIu32 " of memory", percent_to_use); - THANDLE(JOB_OBJECT_HELPER) job_object_helper = job_object_helper_set_job_limits_to_current_process("", 100, percent_to_use); - ASSERT_IS_NOT_NULL(job_object_helper); + int job_object_helper_result = job_object_helper_set_job_limits_to_current_process("", 100, percent_to_use); + ASSERT_ARE_EQUAL(int, 0, job_object_helper_result); already_exists_item = CLDS_HASH_TABLE_NODE_CREATE(TEST_ITEM, NULL, NULL); ASSERT_IS_NOT_NULL(already_exists_item); @@ -162,7 +161,7 @@ TEST_FUNCTION(clds_hash_table_insert_same_block_again_does_not_leak_item_count) CLDS_HASH_TABLE_NODE_RELEASE(TEST_ITEM, already_exists_item); clds_hazard_pointers_destroy(hazard_pointers); - THANDLE_ASSIGN(JOB_OBJECT_HELPER)(&job_object_helper, NULL); + job_object_helper_deinit_for_test(); } END_TEST_SUITE(TEST_SUITE_NAME_FROM_CMAKE)