diff --git a/tools/compliance/policy/policy.go b/tools/compliance/policy/policy.go index 9dab05bbf0..d3e412b84e 100644 --- a/tools/compliance/policy/policy.go +++ b/tools/compliance/policy/policy.go @@ -20,6 +20,15 @@ import ( ) var ( + // RecognizedAnnotations identifies the set of annotations that have + // meaning for compliance policy. + RecognizedAnnotations = map[string]string{ + // used in readgraph.go to avoid creating 1000's of copies of the below 3 strings. + "static": "static", + "dynamic": "dynamic", + "toolchain": "toolchain", + } + // ImpliesUnencumbered lists the condition names representing an author attempt to disclaim copyright. ImpliesUnencumbered = ConditionNames{"unencumbered"} diff --git a/tools/compliance/readgraph.go b/tools/compliance/readgraph.go index 45fa1343ba..face775479 100644 --- a/tools/compliance/readgraph.go +++ b/tools/compliance/readgraph.go @@ -188,10 +188,11 @@ func addDependencies(edges *[]*dependencyEdge, target string, dependencies []*li } annotations := newEdgeAnnotations() for _, a := range ad.Annotations { - if len(a) == 0 { - continue + // look up a common constant annotation string from a small map + // instead of creating 1000's of copies of the same 3 strings. + if ann, ok := RecognizedAnnotations[a]; ok { + annotations.annotations[ann] = true } - annotations.annotations[a] = true } *edges = append(*edges, &dependencyEdge{target, dependency, annotations}) }