chainguard-dev / malcontent

#supply #chain #attack #detection
Apache License 2.0
445 stars 31 forks source link

Replace combined channel with slice #430

Closed egibs closed 2 months ago

egibs commented 2 months ago

Now that combine is not called concurrently, it doesn't make sense to store the combined reports in a channel (combined).

This PR replaces said channel with a slice of bincapz.CombinedReport with a capacity that matches the product of lengths of the two filtered map channels, rather a capacity equal to the sum of the lengths of the unfiltered removed and added maps which the channel was using.