Skip to content

Commit 909a614

Browse files
committed
remove unnecessary cast after malloc-ish functions
it's not cpp.... :) Signed-off-by: Łukasz Plewa <lukasz.plewa@intel.com>
1 parent 9f1fd53 commit 909a614

File tree

4 files changed

+12
-19
lines changed

4 files changed

+12
-19
lines changed

src/memory_target.c

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ umf_result_t umfMemoryTargetCreate(const umf_memory_target_ops_t *ops,
2525
}
2626

2727
umf_memory_target_handle_t target =
28-
(umf_memory_target_t *)umf_ba_global_alloc(sizeof(umf_memory_target_t));
28+
umf_ba_global_alloc(sizeof(umf_memory_target_t));
2929
if (!target) {
3030
return UMF_RESULT_ERROR_OUT_OF_HOST_MEMORY;
3131
}
@@ -59,8 +59,7 @@ umf_result_t umfMemoryTargetClone(umf_memory_target_handle_t memoryTarget,
5959
assert(memoryTarget);
6060
assert(outHandle);
6161

62-
*outHandle =
63-
(umf_memory_target_t *)umf_ba_global_alloc(sizeof(umf_memory_target_t));
62+
*outHandle = umf_ba_global_alloc(sizeof(umf_memory_target_t));
6463
if (!*outHandle) {
6564
return UMF_RESULT_ERROR_OUT_OF_HOST_MEMORY;
6665
}

src/memspaces/memspace_numa.c

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,15 +23,14 @@ umfMemspaceCreateFromNumaArray(size_t *nodeIds, size_t numIds,
2323

2424
enum umf_result_t ret = UMF_RESULT_SUCCESS;
2525
umf_memspace_handle_t memspace =
26-
(struct umf_memspace_t *)umf_ba_global_alloc(
27-
sizeof(struct umf_memspace_t));
26+
umf_ba_global_alloc(sizeof(struct umf_memspace_t));
2827
if (!memspace) {
2928
return UMF_RESULT_ERROR_OUT_OF_HOST_MEMORY;
3029
}
3130

3231
memspace->size = numIds;
33-
memspace->nodes = (umf_memory_target_handle_t *)umf_ba_global_alloc(
34-
memspace->size * sizeof(umf_memory_target_handle_t));
32+
memspace->nodes = umf_ba_global_alloc(memspace->size *
33+
sizeof(umf_memory_target_handle_t));
3534
if (!memspace->nodes) {
3635
ret = UMF_RESULT_ERROR_OUT_OF_HOST_MEMORY;
3736
goto err_nodes_alloc;

src/provider/provider_os_memory.c

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -218,8 +218,7 @@ static umf_result_t os_initialize(void *params, void **provider) {
218218
(umf_os_memory_provider_params_t *)params;
219219

220220
os_memory_provider_t *os_provider =
221-
(os_memory_provider_t *)umf_ba_global_alloc(
222-
sizeof(os_memory_provider_t));
221+
umf_ba_global_alloc(sizeof(os_memory_provider_t));
223222
if (!os_provider) {
224223
return UMF_RESULT_ERROR_OUT_OF_HOST_MEMORY;
225224
}

src/provider/provider_tracking.c

Lines changed: 6 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,7 @@ static umf_result_t umfMemoryTrackerAdd(umf_memory_tracker_handle_t hTracker,
3535
const void *ptr, size_t size) {
3636
assert(ptr);
3737

38-
tracker_value_t *value =
39-
(tracker_value_t *)umf_ba_alloc(hTracker->tracker_allocator);
38+
tracker_value_t *value = umf_ba_alloc(hTracker->tracker_allocator);
4039
if (value == NULL) {
4140
return UMF_RESULT_ERROR_OUT_OF_HOST_MEMORY;
4241
}
@@ -170,7 +169,7 @@ static umf_result_t trackingAllocationSplit(void *hProvider, void *ptr,
170169
(umf_tracking_memory_provider_t *)hProvider;
171170

172171
tracker_value_t *splitValue =
173-
(tracker_value_t *)umf_ba_alloc(provider->hTracker->tracker_allocator);
172+
umf_ba_alloc(provider->hTracker->tracker_allocator);
174173
if (!splitValue) {
175174
return UMF_RESULT_ERROR_OUT_OF_HOST_MEMORY;
176175
}
@@ -244,7 +243,7 @@ static umf_result_t trackingAllocationMerge(void *hProvider, void *lowPtr,
244243
(umf_tracking_memory_provider_t *)hProvider;
245244

246245
tracker_value_t *mergedValue =
247-
(tracker_value_t *)umf_ba_alloc(provider->hTracker->tracker_allocator);
246+
umf_ba_alloc(provider->hTracker->tracker_allocator);
248247

249248
if (!mergedValue) {
250249
return UMF_RESULT_ERROR_OUT_OF_HOST_MEMORY;
@@ -363,8 +362,7 @@ static umf_result_t trackingFree(void *hProvider, void *ptr, size_t size) {
363362

364363
static umf_result_t trackingInitialize(void *params, void **ret) {
365364
umf_tracking_memory_provider_t *provider =
366-
(umf_tracking_memory_provider_t *)umf_ba_global_alloc(
367-
sizeof(umf_tracking_memory_provider_t));
365+
umf_ba_global_alloc(sizeof(umf_tracking_memory_provider_t));
368366
if (!provider) {
369367
return UMF_RESULT_ERROR_OUT_OF_HOST_MEMORY;
370368
}
@@ -511,8 +509,7 @@ static umf_result_t trackingGetIpcHandle(void *provider, const void *ptr,
511509
}
512510

513511
size_t value_size = sizeof(ipc_cache_value_t) + ipcDataSize;
514-
ipc_cache_value_t *cache_value =
515-
(ipc_cache_value_t *)umf_ba_global_alloc(value_size);
512+
ipc_cache_value_t *cache_value = umf_ba_global_alloc(value_size);
516513
if (!cache_value) {
517514
LOG_ERR(
518515
"tracking get ipc handle: failed to allocate cache_value");
@@ -676,8 +673,7 @@ void umfTrackingMemoryProviderGetUpstreamProvider(
676673

677674
umf_memory_tracker_handle_t umfMemoryTrackerCreate(void) {
678675
umf_memory_tracker_handle_t handle =
679-
(umf_memory_tracker_handle_t)umf_ba_global_alloc(
680-
sizeof(struct umf_memory_tracker_t));
676+
umf_ba_global_alloc(sizeof(struct umf_memory_tracker_t));
681677
if (!handle) {
682678
return NULL;
683679
}

0 commit comments

Comments
 (0)