Merge "Remove `OptionalImplicitSdkLibrary()` function (it's not used anymore)." am: c2427ea290 (a9692035) · Commits · e / os / android_build_soong

java/sdk_library.go

+8 −12

Original line number	Diff line number	Diff line
		@@ -980,14 +980,16 @@ func (e EmbeddableSdkLibraryComponent) SdkLibraryName() string {
		return e.sdkLibraryComponentProperties.SdkLibraryName
		}

		// to satisfy SdkLibraryComponentDependency
		func (e EmbeddableSdkLibraryComponent) OptionalImplicitSdkLibrary() string {
		return e.sdkLibraryComponentProperties.SdkLibraryToImplicitlyTrack
		}

		// to satisfy SdkLibraryComponentDependency
		func (e EmbeddableSdkLibraryComponent) OptionalSdkLibraryImplementation() string {
		// Currently implementation library name is the same as the SDK library name.
		// For shared libraries, this is the same as the SDK library name. If a Java library or app
		// depends on a component library (e.g. a stub library) it still needs to know the name of the
		// run-time library and the corresponding module that provides the implementation. This name is
		// passed to manifest_fixer (to be added to AndroidManifest.xml) and added to CLC (to be used
		// in dexpreopt).
		//
		// For non-shared SDK (component or not) libraries this returns `nil`, as they are not
		// <uses-library> and should not be added to the manifest or to CLC.
		return e.sdkLibraryComponentProperties.SdkLibraryToImplicitlyTrack
		}

		@@ -999,12 +1001,6 @@ type SdkLibraryComponentDependency interface {
		// SdkLibraryName returns the name of the java_sdk_library/_import module.
		SdkLibraryName() *string

		// The optional name of the sdk library that should be implicitly added to the
		// AndroidManifest of an app that contains code which references the sdk library.
		//
		// Returns the name of the optional implicit SDK library or nil, if there isn't one.
		OptionalImplicitSdkLibrary() *string

		// The name of the implementation library for the optional SDK library or nil, if there isn't one.
		OptionalSdkLibraryImplementation() *string
		}