Fix style comments

Author: Sarthak Agrawal

internal/controller/state/graph/graph_test.go

@@ -795,7 +795,7 @@ func TestBuildGraph(t *testing.T) {
 						): {"fizz.example.org"},
 					},
 				},
-				SectionName: nil, // No expansion - represents attachment to all listeners
+				SectionName: nil, // represents attachment to all listeners
 			},
 		},
 		Spec: L4RouteSpec{

internal/controller/state/graph/policies.go

@@ -462,7 +462,7 @@ func checkForRouteOverlap(route *L7Route, hostPortPaths map[string]string) *cond
 		port := parentRef.Attachment.ListenerPort
 		for _, hostnames := range parentRef.Attachment.AcceptedHostnames {
 			mergedHostnames := getMergedHostnames(route, hostnames)
-			if cond := checkHostnamesOverlap(route, mergedHostnames, port, hostPortPaths); cond != nil {
+			if cond := checkURIsOverlap(route, mergedHostnames, port, hostPortPaths); cond != nil {
 				return cond
 			}
 		}
@@ -483,47 +483,36 @@ func getMergedHostnames(route *L7Route, hostnames []string) []string {
 }
 
 func getHostnameMatches(hostname string, routeHostnames []v1.Hostname) []string {
-	foundExactMatch := false
 	moreSpecificHostnames := make([]string, 0)
 
 	for _, h := range routeHostnames {
 		if hostname == string(h) {
-			foundExactMatch = true
-			break
+			return []string{hostname}
 		}
-		if Match(hostname, string(h)) {
+		if match(hostname, string(h)) {
 			moreSpecificHostnames = append(moreSpecificHostnames, GetMoreSpecificHostname(hostname, string(h)))
 		}
 	}
 
-	if foundExactMatch {
-		return []string{hostname}
-	}
 	return moreSpecificHostnames
 }
 
-func checkHostnamesOverlap(
-	route *L7Route, hostnames []string, port v1.PortNumber, hostPortPaths map[string]string,
+func checkURIsOverlap(
+	route *L7Route,
+	hostnames []string,
+	port v1.PortNumber,
+	hostPortPaths map[string]string,
 ) *conditions.Condition {
 	for _, hostname := range hostnames {
-		if cond := checkHostnamePathsOverlap(route, hostname, port, hostPortPaths); cond != nil {
-			return cond
-		}
-	}
-	return nil
-}
-
-func checkHostnamePathsOverlap(
-	route *L7Route, hostname string, port v1.PortNumber, hostPortPaths map[string]string,
-) *conditions.Condition {
-	for _, rule := range route.Spec.Rules {
-		for _, match := range rule.Matches {
-			if match.Path != nil && match.Path.Value != nil {
-				key := fmt.Sprintf("%s:%d%s", hostname, port, *match.Path.Value)
-				if val, ok := hostPortPaths[key]; !ok {
-					hostPortPaths[key] = fmt.Sprintf("%s/%s", route.Source.GetNamespace(), route.Source.GetName())
-				} else {
-					return createTargetConflictCondition(route, val)
+		for _, rule := range route.Spec.Rules {
+			for _, match := range rule.Matches {
+				if match.Path != nil && match.Path.Value != nil {
+					key := fmt.Sprintf("%s:%d%s", hostname, port, *match.Path.Value)
+					if val, ok := hostPortPaths[key]; !ok {
+						hostPortPaths[key] = fmt.Sprintf("%s/%s", route.Source.GetNamespace(), route.Source.GetName())
+					} else {
+						return createTargetConflictCondition(route, val)
+					}
 				}
 			}
 		}

internal/controller/state/graph/route_common.go

@@ -318,12 +318,10 @@ func buildSectionNameRefs(
 				}
 			}
 
-			// Create a single parentRef without expansion
 			sectionNameRefs = append(sectionNameRefs, ParentRef{
-				Idx:         i,
-				Gateway:     CreateParentRefGateway(gw),
-				SectionName: nil, // Keep as nil to indicate attachment to multiple listeners
-				Port:        p.Port,
+				Idx:     i,
+				Gateway: CreateParentRefGateway(gw),
+				Port:    p.Port,
 			})
 
 			continue
@@ -926,15 +924,15 @@ func findAcceptedHostnames(listenerHostname *v1.Hostname, routeHostnames []v1.Ho
 
 	for _, h := range routeHostnames {
 		routeHost := string(h)
-		if Match(hostname, routeHost) {
+		if match(hostname, routeHost) {
 			result = append(result, GetMoreSpecificHostname(hostname, routeHost))
 		}
 	}
 
 	return result
 }
 
-func Match(listenerHost, routeHost string) bool {
+func match(listenerHost, routeHost string) bool {
 	if listenerHost == "" {
 		return true
 	}