On Wed, Mar 20, 2024 at 7:55 AM Kuan-Wei Chiu <visitorckw@xxxxxxxxx> wrote: > > Replace direct assignment of values to heap data members with > min_heap_init() for better code readability and maintainability. > > Link: https://lkml.kernel.org/CAP-5=fW+FvUu8JL+KrtVv5uC++4AW=VhyEOgmdWzpH1mswQNzw@xxxxxxxxxxxxxx > Signed-off-by: Kuan-Wei Chiu <visitorckw@xxxxxxxxx> Ah, got it :-) Reviewed-by: Ian Rogers <irogers@xxxxxxxxxx> Thanks, Ian > --- > lib/test_min_heap.c | 11 +++-------- > 1 file changed, 3 insertions(+), 8 deletions(-) > > diff --git a/lib/test_min_heap.c b/lib/test_min_heap.c > index 062e908e9fa3..8d25fc8256db 100644 > --- a/lib/test_min_heap.c > +++ b/lib/test_min_heap.c > @@ -67,9 +67,8 @@ static __init int test_heapify_all(bool min_heap) > -3, -1, -2, -4, 0x8000000, 0x7FFFFFF }; > struct min_heap_test heap; > > - heap.heap.data = values; > + min_heap_init(&heap, values, ARRAY_SIZE(values)); > heap.heap.nr = ARRAY_SIZE(values); > - heap.heap.size = ARRAY_SIZE(values); > struct min_heap_callbacks funcs = { > .less = min_heap ? less_than : greater_than, > .swp = swap_ints, > @@ -99,9 +98,7 @@ static __init int test_heap_push(bool min_heap) > int values[ARRAY_SIZE(data)]; > struct min_heap_test heap; > > - heap.heap.data = values; > - heap.heap.nr = 0; > - heap.heap.size = ARRAY_SIZE(values); > + min_heap_init(&heap, values, ARRAY_SIZE(values)); > struct min_heap_callbacks funcs = { > .less = min_heap ? less_than : greater_than, > .swp = swap_ints, > @@ -131,9 +128,7 @@ static __init int test_heap_pop_push(bool min_heap) > int values[ARRAY_SIZE(data)]; > struct min_heap_test heap; > > - heap.heap.data = values; > - heap.heap.nr = 0; > - heap.heap.size = ARRAY_SIZE(values); > + min_heap_init(&heap, values, ARRAY_SIZE(values)); > struct min_heap_callbacks funcs = { > .less = min_heap ? less_than : greater_than, > .swp = swap_ints, > -- > 2.34.1 >