Re: [PATCH v5 0/1] MR2622: ntdll: Use log-linear bucketing for free lists.

[Tatsuyuki Ishi (＠ishitatsuyuki)]

On Tue Apr 11 07:23:05 2023 +0000, Rémi Bernon wrote:
> ```suggestion:-10+0
> /* locate a free list entry of the appropriate size */
> static inline struct entry *find_free_list( struct heap *heap, SIZE_T
> block_size, BOOL last )
> {
>     UINT index = get_free_list_index( block_size );
>     if (last && ++index == FREE_LIST_COUNT) index = 0;
>     return &heap->free_lists[index];
> }
> ```
> I think we can remove the block_size comments at this point,
> `block_size` is used consistently now and implies that the size includes
> the block header.
> I'd also prefer to use "index" rather than "block" which already has a
> different meaning.

Thanks, done.

-- 
https://gitlab.winehq.org/wine/wine/-/merge_requests/2622#note_29555