# [PATCH] t/t0211-trace2-perf.sh: fix typo patern -> pattern

3 messages from 2024-05-15 to 2024-05-16. Participants: Marcel Telka, Patrick Steinhardt, Junio C Hamano.
Thread: https://gitlist.dev/t/61485

## Marcel Telka, 2024-05-15 22:51

Subject: [PATCH] t/t0211-trace2-perf.sh: fix typo patern -> pattern
Message-ID: <ZkU8VAUnKx_SXU5U@telcontar>
URL: https://gitlist.dev/e/ZkU8VAUnKx_SXU5U%40telcontar

```
Signed-off-by: Marcel Telka <marcel@telka.sk>
---
 t/t0211-trace2-perf.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/t/t0211-trace2-perf.sh b/t/t0211-trace2-perf.sh
index 13ef69b92f..070fe7a5da 100755
--- a/t/t0211-trace2-perf.sh
+++ b/t/t0211-trace2-perf.sh
@@ -233,7 +233,7 @@ have_counter_event () {
 
 	pattern="d0|${thread}|${event}||||${category}|name:${name} value:${value}" &&
 
-	grep "${patern}" ${file}
+	grep "${pattern}" ${file}
 }
 
 test_expect_success 'global counter test/test1' '

```

## Patrick Steinhardt, 2024-05-16 04:17

Subject: Re: [PATCH] t/t0211-trace2-perf.sh: fix typo patern -> pattern
Message-ID: <ZkWI9--gMH0Xgv3f@tanuki>
URL: https://gitlist.dev/e/ZkWI9--gMH0Xgv3f%40tanuki
In-Reply-To: <ZkU8VAUnKx_SXU5U@telcontar>

```
On Thu, May 16, 2024 at 12:51:00AM +0200, Marcel Telka wrote:

Thanks for the patch, which looks obviously good to me. It would be nice
to have a short explanation in the commit message that explains why this
issue was able to sneak.

Patrick

> Signed-off-by: Marcel Telka <marcel@telka.sk>
> ---
>  t/t0211-trace2-perf.sh | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/t/t0211-trace2-perf.sh b/t/t0211-trace2-perf.sh
> index 13ef69b92f..070fe7a5da 100755
> --- a/t/t0211-trace2-perf.sh
> +++ b/t/t0211-trace2-perf.sh
> @@ -233,7 +233,7 @@ have_counter_event () {
>  
>  	pattern="d0|${thread}|${event}||||${category}|name:${name} value:${value}" &&
>  
> -	grep "${patern}" ${file}
> +	grep "${pattern}" ${file}
>  }
>  
>  test_expect_success 'global counter test/test1' '
> 

```

## Junio C Hamano, 2024-05-16 15:38

Subject: Re: [PATCH] t/t0211-trace2-perf.sh: fix typo patern -> pattern
Message-ID: <xmqq34qhu5qo.fsf@gitster.g>
URL: https://gitlist.dev/e/xmqq34qhu5qo.fsf%40gitster.g
In-Reply-To: <ZkWI9--gMH0Xgv3f@tanuki>

```
Patrick Steinhardt <ps@pks.im> writes:

> On Thu, May 16, 2024 at 12:51:00AM +0200, Marcel Telka wrote:
>
> Thanks for the patch, which looks obviously good to me. It would be nice
> to have a short explanation in the commit message that explains why this
> issue was able to sneak.
>
> Patrick

A good piece of advice.

FYI, I have the following since last night, as the original report
was whitespace damaged and did not apply.

-------- >8 --------------- >8 --------------- >8 --------
Subject: [PATCH] t0211: typofix

Marcel Telka noticed that one test assigns to $pattern and then
uses $patern in an invocation of "grep".  Of course, looking for
an empty string will succeed, so the test succeeds for a wrong
reason.

Signed-off-by: Junio C Hamano <gitster@pobox.com>
---
 t/t0211-trace2-perf.sh | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/t/t0211-trace2-perf.sh b/t/t0211-trace2-perf.sh
index 0b3436e8ca..37a51e6449 100755
--- a/t/t0211-trace2-perf.sh
+++ b/t/t0211-trace2-perf.sh
@@ -233,7 +233,7 @@ have_counter_event () {
 
 	pattern="d0|${thread}|${event}||||${category}|name:${name} value:${value}" &&
 
-	grep "${patern}" ${file}
+	grep "${pattern}" ${file}
 }
 
 test_expect_success 'global counter test/test1' '
-- 
2.45.1-190-g19fe900cfc

```