Delete unused variables and arguments

Author: keith

mobile_install/apks.bzl

@@ -115,7 +115,7 @@ def make_split_apks(
     to_pack = dexes + [r_dex]
     if native_zips:
         to_pack.extend(native_zips)
-    for i, artifact in enumerate(to_pack):
+    for artifact in to_pack:
         # the split attr in the manifest will be used to name the file on the device like
         # split_${SPLIT_ID}.apk. We need to follow the same pattern so that we can compare
         # files during sync time and only do the incremental install.

rules/aar_import/impl.bzl

@@ -51,10 +51,6 @@ RULE_PREFIX = "_aar"
 ANDROID_MANIFEST = "AndroidManifest.xml"
 LINT_JAR = "lint.jar"
 
-# Resources context dict fields.
-_PROVIDERS = "providers"
-_VALIDATION_RESULTS = "validation_results"
-
 def _create_aar_tree_artifact(ctx, name):
     return ctx.actions.declare_directory("%s/unzipped/%s/%s" % (RULE_PREFIX, name, ctx.label.name))
 
@@ -138,9 +134,7 @@ def _process_resources(
         aar,
         package,
         manifest,
-        deps,
-        aar_resources_extractor_tool,
-        unzip_tool):
+        aar_resources_extractor_tool):
     # Extract resources and assets, if they exist.
     resources = _create_aar_tree_artifact(ctx, "resources")
     assets = _create_aar_tree_artifact(ctx, "assets")
@@ -492,10 +486,8 @@ def impl(ctx):
         aar = aar,
         package = package,
         manifest = manifest_ctx.processed_manifest,
-        deps = ctx.attr.deps,
         aar_resources_extractor_tool =
             _get_android_toolchain(ctx).aar_resources_extractor.files_to_run,
-        unzip_tool = unzip_tool,
     )
     providers.extend(resources_ctx.providers)
 

rules/android_application/android_application_rule.bzl

@@ -95,8 +95,6 @@ def _process_feature_module(
         get_android_sdk(ctx).aapt2,
         ctx.executable._feature_manifest_script,
         ctx.executable._priority_feature_manifest_script,
-        get_android_toolchain(ctx).android_resources_busybox,
-        _common.get_host_javabase(ctx),
     )
 
     # Remove all dexes from the feature module apk. jvm / resources are not
@@ -186,9 +184,7 @@ def _create_feature_manifest(
         feature_target,
         aapt2,
         feature_manifest_script,
-        priority_feature_manifest_script,
-        android_resources_busybox,
-        host_javabase):
+        priority_feature_manifest_script):
     info = feature_target[AndroidFeatureModuleInfo]
     manifest = ctx.actions.declare_file(ctx.label.name + "/" + feature_target.label.name + "/AndroidManifest.xml")
 

rules/android_binary/impl.bzl

@@ -259,7 +259,7 @@ def _process_build_info(_unused_ctx, **unused_ctxs):
         ),
     )
 
-def _process_dex(ctx, validation_ctx, packaged_resources_ctx, manifest_ctx, deploy_ctx, bp_ctx, optimize_ctx, **_unused_ctxs):
+def _process_dex(ctx, validation_ctx, packaged_resources_ctx, manifest_ctx, deploy_ctx, optimize_ctx, **_unused_ctxs):
     if validation_ctx.use_r8:
         return ProviderInfo(
             name = "dex_ctx",
@@ -421,7 +421,7 @@ def _process_dex(ctx, validation_ctx, packaged_resources_ctx, manifest_ctx, depl
         ),
     )
 
-def _process_deploy_jar(ctx, validation_ctx, stamp_ctx, manifest_ctx, packaged_resources_ctx, jvm_ctx, build_info_ctx, proto_ctx, **_unused_ctxs):
+def _process_deploy_jar(ctx, validation_ctx, stamp_ctx, manifest_ctx, jvm_ctx, build_info_ctx, **_unused_ctxs):
     if validation_ctx.use_r8:
         return ProviderInfo(
             name = "deploy_ctx",
@@ -608,17 +608,6 @@ def _get_library_r_jars(deps):
         transitive_resource_jars += dep.jars.to_list()
     return transitive_resource_jars
 
-def _is_test_binary(ctx):
-    """Whether this android_binary target is a test binary.
-
-    Args:
-      ctx: The context.
-
-    Returns:
-      Boolean indicating whether the target is a test target.
-    """
-    return ctx.attr.testonly or is_instrumentation(ctx) or str(ctx.label).find("/javatests/") >= 0
-
 def is_instrumentation(ctx):
     """Whether this android_binary target is an instrumentation binary.
 

rules/android_binary/rule.bzl

@@ -27,8 +27,6 @@ load(":impl.bzl", "impl")
 
 visibility(PROJECT_VISIBILITY)
 
-_DEFAULT_ALLOWED_ATTRS = ["name", "visibility", "tags", "testonly", "transitive_configs", "$enable_manifest_merging", "features", "exec_properties"]
-
 _DEFAULT_PROVIDES = [ApkInfo, JavaInfo]
 
 def _outputs(name, proguard_generate_mapping, _package_name, _generate_proguard_outputs, _generate_art_profile_outputs):
@@ -126,7 +124,6 @@ def android_binary_macro(**attrs):
 
     # Required for ACLs check in _outputs(), since the callback can't access the native module.
     attrs["$package_name"] = native.package_name()
-    target_fqn = "//%s:%s" % (native.package_name(), attrs["name"])
 
     if type(attrs.get("proguard_specs", None)) == "select" or attrs.get("proguard_specs", None):
         attrs["$generate_proguard_outputs"] = True

rules/android_library/impl.bzl

@@ -318,7 +318,7 @@ def _process_lint_rules(ctx, **unused_sub_ctxs):
         ),
     )
 
-def _process_aar(ctx, java_package, resources_ctx, proguard_ctx, **unused_ctx):
+def _process_aar(ctx, resources_ctx, proguard_ctx, **unused_ctx):
     aar_ctx = {
         _PROVIDERS: [],
         _VALIDATION_OUTPUTS: [],
@@ -460,7 +460,6 @@ PROCESSORS = dict(
 def finalize(
         ctx,
         resources_ctx,
-        intellij_ctx,
         jvm_ctx,
         proguard_ctx,
         providers,

rules/data_binding.bzl

@@ -132,7 +132,6 @@ def _setup_dependent_lib_artifacts(ctx, output_dir, deps):
     return dep_lib_artifacts
 
 def _get_javac_opts(
-        ctx,
         java_package,
         artifact_type,
         dependency_artifacts_dir,
@@ -284,7 +283,6 @@ def _process(
     )
 
     db_info[_JAVAC_OPTS] = _get_javac_opts(
-        ctx,
         java_package,
         artifact_type,
         (

rules/dex_desugar_aspect.bzl

@@ -120,7 +120,7 @@ def _aspect_impl(target, ctx):
 
     if runtime_jars:
         basename_clash = _check_basename_clash(runtime_jars)
-        aspect_dexopts = _get_aspect_dexopts(ctx)
+        aspect_dexopts = _get_aspect_dexopts()
         for jar in runtime_jars:
             if ctx.fragments.android.desugar_java8:
                 jar_to_desugar = jar
@@ -214,7 +214,7 @@ def _get_platform_based_toolchain_jars(ctx):
         return ctx.rule.attr._aidl_lib[JavaInfo].runtime_output_jars
     return []
 
-def _get_aspect_dexopts(ctx):
+def _get_aspect_dexopts():
     return _power_set(_dex.normalize_dexopts(_dex.DEXOPTS_SUPPORTED_IN_INCREMENTAL_DEXING))
 
 def _get_boot_classpath(target, ctx):

rules/min_sdk_version.bzl

@@ -18,15 +18,15 @@ it holds utilities for handling minSdkVersion propagation.
 
 """
 
-load("//rules:visibility.bzl", "PROJECT_VISIBILITY")
 load("@bazel_skylib//rules:common_settings.bzl", "BuildSettingInfo")
+load("//rules:visibility.bzl", "PROJECT_VISIBILITY")
 
 visibility(PROJECT_VISIBILITY)
 
 _SETTING = "//rules/flags:min_sdk_version"
 _DEPOT_FLOOR = 23
 _DEX_DESUGAR = 23  # Kept at 23 due to b/331852939. See some related discussion at b/319114605
-_MIN_SDK_LEVELS = sorted([_DEPOT_FLOOR, 24])
+# _MIN_SDK_LEVELS = sorted([_DEPOT_FLOOR, 24])
 
 _ATTRS = dict(
     _min_sdk_version = attr.label(
@@ -35,7 +35,7 @@ _ATTRS = dict(
     ),
 )
 
-def _clamp(min_sdk_version):
+def _clamp(min_sdk_version):  # @unused
     # TODO(asinclair): Uncomment this once android_binary is Starlarkified and the order of the
     # Android Platforms Transition and Feature Flags transition is swapped.
     # clamped = _MIN_SDK_LEVELS[0]
@@ -46,7 +46,7 @@ def _clamp(min_sdk_version):
     # return clamped
     return 0
 
-def _get(ctx):
+def _get(ctx):  # @unused
     # This is the case when an android_binary target does not set a value explicitly.
     # The configuration value defaults to 0
     # So in this case we use the depot floor.

rules/native_deps.bzl

@@ -35,7 +35,6 @@ SplitConfigInfo = provider(
 )
 
 def _split_config_aspect_impl(__, ctx):
-    android_cfg = ctx.fragments.android
     return SplitConfigInfo(
         build_config = ctx.configuration,
         target_platform = ctx.fragments.platform.platform,