From edd61643be3096af46d1f6aea1091897f668c66b Mon Sep 17 00:00:00 2001 From: Anton Hansson Date: Fri, 4 Jun 2021 13:54:28 +0100 Subject: [PATCH] Stop filtering html files from droidstubs input Metalava now expects html files to be specified explicitly. This filtering was originally added because jdiff could not cope with html inputs, but jdiff support has been removed in r.android.com/1455335. Bug: 153703940 Test: m checkapi Change-Id: I34164b633bab2892e3a1c0fd0257fcf98d537655 --- java/droiddoc.go | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/java/droiddoc.go b/java/droiddoc.go index 56e62473f..869a5982d 100644 --- a/java/droiddoc.go +++ b/java/droiddoc.go @@ -425,23 +425,6 @@ func (j *Javadoc) collectDeps(ctx android.ModuleContext) deps { } srcFiles = filterByPackage(srcFiles, j.properties.Filter_packages) - // While metalava needs package html files, it does not need them to be explicit on the command - // line. javadoc complains if it receives html files on the command line. The filter - // below excludes html files from the rsp file metalava. Note that the html - // files are still included as implicit inputs for successful remote execution and correct - // incremental builds. - filterHtml := func(srcs []android.Path) []android.Path { - filtered := []android.Path{} - for _, src := range srcs { - if src.Ext() == ".html" { - continue - } - filtered = append(filtered, src) - } - return filtered - } - srcFiles = filterHtml(srcFiles) - aidlFlags := j.collectAidlFlags(ctx, deps) srcFiles = j.genSources(ctx, srcFiles, aidlFlags)