Re: [PATCH v7 0/8] MR1628: ntdll: Implement Low Fragmentation Heap.

[Jinoh Kang (＠iamahuman)]

Jinoh Kang (@iamahuman) commented about dlls/ntdll/heap.c:
> - SIZE_T offset = (char *)block - (char *)&group->first_block;
> - block->base_offset = offset / block_size;
>
> +}
> -
>
> +static inline struct block *group_get_block( struct group *group, SIZE_T block_size, UINT index )
> +{
> - char *first_block = (char *)&group->first_block;
> - return (struct block *)(first_block + index * block_size);
>
> +}
> -
>
> +/* lookup a free block using the group free_bits, the current thread must own the group */
> +static inline LONG group_find_free_block( struct group *group, SIZE_T block_size, struct block **block )
> +{
> - ULONG i, free_bits = ReadNoFence( &group->free_bits );
> - /* free_bits & ~GROUP_FLAG_FREE will never be 0 as the group is unlinked when it's fully used */
> - BitScanForward( &i, free_bits & ~GROUP_FLAG_FREE );

`[free-unset-v7]`
```suggestion:-1+0
    /* free_bits will never be 0 as the group is unlinked when it's fully used */
    BitScanForward( &i, free_bits );
```