2023-07-20 15:59:51

by pangzizhen001

[permalink] [raw]
Subject: [PATCH] zstd/decompress: Fix typos in comments

Delete duplicate word "the"

Signed-off-by: Zizhen Pang <[email protected]>
---
lib/zstd/decompress/zstd_decompress_block.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/lib/zstd/decompress/zstd_decompress_block.c
b/lib/zstd/decompress/zstd_decompress_block.c
index c1913b8e7c89..cd2e9acecd84 100644
--- a/lib/zstd/decompress/zstd_decompress_block.c
+++ b/lib/zstd/decompress/zstd_decompress_block.c
@@ -510,7 +510,7 @@ void ZSTD_buildFSETable_body(ZSTD_seqSymbol* dt,
}
}
/* Now we spread those positions across the table.
- * The benefit of doing it in two stages is that we avoid the
the
+ * The benefit of doing it in two stages is that we avoid the
* variable size inner loop, which caused lots of branch
misses.
* Now we can run through all the positions without any branch
misses.
* We unroll the loop twice, since that is what emperically
worked best.


2023-10-12 16:24:31

by Nick Terrell

[permalink] [raw]
Subject: Re: [PATCH] zstd/decompress: Fix typos in comments



> On Jul 20, 2023, at 11:34 AM, [email protected] wrote:
>
> !-------------------------------------------------------------------|
> This Message Is From an External Sender
>
> |-------------------------------------------------------------------!
>
> Delete duplicate word "the"

Please submit this patch upstream to https://github.com/facebook/zstd if you would like to see it merged.

Thanks,
Nick Terrell

> Signed-off-by: Zizhen Pang <[email protected]>
> ---
> lib/zstd/decompress/zstd_decompress_block.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/lib/zstd/decompress/zstd_decompress_block.c b/lib/zstd/decompress/zstd_decompress_block.c
> index c1913b8e7c89..cd2e9acecd84 100644
> --- a/lib/zstd/decompress/zstd_decompress_block.c
> +++ b/lib/zstd/decompress/zstd_decompress_block.c
> @@ -510,7 +510,7 @@ void ZSTD_buildFSETable_body(ZSTD_seqSymbol* dt,
> }
> }
> /* Now we spread those positions across the table.
> - * The benefit of doing it in two stages is that we avoid the the
> + * The benefit of doing it in two stages is that we avoid the
> * variable size inner loop, which caused lots of branch misses.
> * Now we can run through all the positions without any branch misses.
> * We unroll the loop twice, since that is what emperically worked best.


Attachments:
winmail.dat (13.13 kB)