refactor: remove some uneeded type checks on pointers to interface

In InferAcceptHeaderFromType we're simply check if the type
implements the interface. It's not needed (nor correct) to check
if the type is a pointer to interface as interfaces are already
references

serialization.go

@@ -349,21 +349,11 @@ func InferAcceptHeaderFromType(ans any) string {
 		return "text/html"
 	}
 
-	_, ok = any(&ans).(*CtxRenderer)
-	if ok {
-		return "text/html"
-	}
-
 	_, ok = ans.(Renderer)
 	if ok {
 		return "text/html"
 	}
 
-	_, ok = any(&ans).(*Renderer)
-	if ok {
-		return "text/html"
-	}
-
 	return "application/json"
 }
 

serialization_test.go

@@ -493,10 +493,26 @@ func TestInferAcceptHeaderFromType(t *testing.T) {
 		require.Equal(t, "text/html", accept)
 	})
 
+	t.Run("can infer that reference type is a template (implements Renderer)", func(t *testing.T) {
+		accept := InferAcceptHeaderFromType(&templateMock{})
+		require.Equal(t, "text/html", accept)
+	})
+
 	t.Run("can infer that type is a template (implements CtxRenderer)", func(t *testing.T) {
 		accept := InferAcceptHeaderFromType(MockCtxRenderer{})
 		require.Equal(t, "text/html", accept)
 	})
+
+	t.Run("can infer that reference type is a template (implements CtxRenderer)", func(t *testing.T) {
+		accept := InferAcceptHeaderFromType(&MockCtxRenderer{})
+		require.Equal(t, "text/html", accept)
+	})
+
+	t.Run("can infer string reference", func(t *testing.T) {
+		s := "hello"
+		accept := InferAcceptHeaderFromType(&s)
+		require.Equal(t, "text/plain", accept)
+	})
 }
 
 func TestInferAcceptHeader(t *testing.T) {