Remove legacy methods Package#{getRules,getFiles}.

--
MOS_MIGRATED_REVID=91573102

1 files changed, 2 insertions, 17 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/packages/Package.java b/src/main/java/com/google/devtools/build/lib/packages/Package.java
index b7ef184f74..48b3e8966c 100644
--- a/src/main/java/com/google/devtools/build/lib/packages/Package.java
+++ b/src/main/java/com/google/devtools/build/lib/packages/Package.java
@@ -524,22 +524,6 @@ public class Package implements Serializable {
   }
 
   /**
-   * Returns a (read-only, unordered) iterator over the rules in this package.
-   */
-  @VisibleForTesting // Legacy.  Production code should use getTargets(Class) instead
-  Iterable<? extends Rule> getRules() {
-    return getTargets(Rule.class);
-  }
-
-  /**
-   * Returns a (read-only, unordered) iterator over the files in this package.
-   */
-  @VisibleForTesting // Legacy.  Production code should use getTargets(Class) instead
-  Iterable<? extends FileTarget> getFiles() {
-    return getTargets(FileTarget.class);
-  }
-
-  /**
    * Returns the rule that corresponds to a particular BUILD target name. Useful
    * for walking through the dependency graph of a target.
    * Fails if the target is not a Rule.
@@ -708,7 +692,8 @@ public class Package implements Serializable {
 
   @Override
   public String toString() {
-    return "Package(" + name + ")=" + (targets != null ? getRules() : "initializing...");
+    return "Package(" + name + ")="
+        + (targets != null ? getTargets(Rule.class) : "initializing...");
   }
 
   /**