Remove legacy methods Package#{getRules,getFiles}.
--
MOS_MIGRATED_REVID=91573102
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 b7ef184..48b3e89 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 @@
}
/**
- * 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 @@
@Override
public String toString() {
- return "Package(" + name + ")=" + (targets != null ? getRules() : "initializing...");
+ return "Package(" + name + ")="
+ + (targets != null ? getTargets(Rule.class) : "initializing...");
}
/**
