From d4e70101500b43ffe705d4c45e50dd4f1c8e3b2e Mon Sep 17 00:00:00 2001 From: Alan Donovan Date: Thu, 2 Apr 2015 10:00:08 -0400 Subject: [PATCH] go/pointer: add TODO comment Change-Id: Ic383036a6b3aa092aa571694bba86d821428f236 Reviewed-on: https://go-review.googlesource.com/8385 Reviewed-by: David Crawshaw --- go/pointer/api.go | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/go/pointer/api.go b/go/pointer/api.go index a81c586d..8f9ae0ab 100644 --- a/go/pointer/api.go +++ b/go/pointer/api.go @@ -21,6 +21,12 @@ type Config struct { // Mains contains the set of 'main' packages to analyze // Clients must provide the analysis with at least one // package defining a main() function. + // + // Non-main packages in the ssa.Program that are not + // dependencies of any main package may still affect the + // analysis result, because they contribute runtime types and + // thus methods. + // TODO(adonovan): investigate whether this is desirable. Mains []*ssa.Package // Reflection determines whether to handle reflection