Remove unnecessary wrapping of function call

Author: withshubh

.deepsource.toml

@@ -101,10 +101,8 @@ type ValueBinder struct {
 // QueryParamsBinder creates query parameter value binder
 func QueryParamsBinder(c Context) *ValueBinder {
 	return &ValueBinder{
-		failFast: true,
-		ValueFunc: func(sourceParam string) string {
-			return c.QueryParam(sourceParam)
-		},
+		failFast:  true,
+		ValueFunc: c.QueryParam,
 		ValuesFunc: func(sourceParam string) []string {
 			values, ok := c.QueryParams()[sourceParam]
 			if !ok {
@@ -119,10 +117,8 @@ func QueryParamsBinder(c Context) *ValueBinder {
 // PathParamsBinder creates path parameter value binder
 func PathParamsBinder(c Context) *ValueBinder {
 	return &ValueBinder{
-		failFast: true,
-		ValueFunc: func(sourceParam string) string {
-			return c.Param(sourceParam)
-		},
+		failFast:  true,
+		ValueFunc: c.Param,
 		ValuesFunc: func(sourceParam string) []string {
 			// path parameter should not have multiple values so getting values does not make sense but lets not error out here
 			value := c.Param(sourceParam)

binder.go

@@ -263,6 +263,4 @@ func (store *RateLimiterMemoryStore) cleanupStaleVisitors() {
 /*
 actual time method which is mocked in test file
 */
-var now = func() time.Time {
-	return time.Now()
-}
+var now = time.Now

middleware/rate_limiter.go

@@ -350,9 +350,7 @@ func TestRateLimiterMemoryStore_Allow(t *testing.T) {
 
 func TestRateLimiterMemoryStore_cleanupStaleVisitors(t *testing.T) {
 	var inMemoryStore = NewRateLimiterMemoryStoreWithConfig(RateLimiterMemoryStoreConfig{Rate: 1, Burst: 3})
-	now = func() time.Time {
-		return time.Now()
-	}
+	now = time.Now
 	fmt.Println(now())
 	inMemoryStore.visitors = map[string]*Visitor{
 		"A": {