Commits on Nov 11, 2023

1. Avoid allocations with (*regexp.Regexp).MatchString …

   We should use `(*regexp.Regexp).MatchString` instead of
   `(*regexp.Regexp).Match([]byte(...))` when matching string to avoid
   unnecessary `[]byte` conversions and reduce allocations.
   
   Example benchmark:
   
   var goTestRegExp = regexp.MustCompile(`_test\.go$`)
   
   func BenchmarkMatch(b *testing.B) {
   	for i := 0; i < b.N; i++ {
   		if match := goTestRegExp.Match([]byte("file_test.go")); !match {
   			b.Fail()
   		}
   	}
   }
   
   func BenchmarkMatchString(b *testing.B) {
   	for i := 0; i < b.N; i++ {
   		if match := goTestRegExp.MatchString("file_test.go"); !match {
   			b.Fail()
   		}
   	}
   }
   
   goos: linux
   goarch: amd64
   pkg: github.com/onsi/ginkgo/v2/ginkgo/watch
   cpu: AMD Ryzen 7 PRO 4750U with Radeon Graphics
   BenchmarkMatch-16          	 5665784	       314.4 ns/op	      16 B/op	       1 allocs/op
   BenchmarkMatchString-16    	 8481872	       140.5 ns/op	       0 B/op	       0 allocs/op
   PASS
   ok  	github.com/onsi/ginkgo/v2/ginkgo/watch	4.321s
   
   Signed-off-by: Eng Zer Jun <engzerjun@gmail.com>

   

   Juneezee committed Nov 11, 2023
   Configuration menu

   • View commit details
   • Copy full SHA for b84f660
   • Browse repository at this point

   Copy the full SHA

   b84f660 View commit details

   Browse the repository at this point in the history