diff options
| author | 2023-03-02 00:56:06 +0000 | |
|---|---|---|
| committer | 2023-03-23 19:38:56 +0000 | |
| commit | a26eda7f2ced8c5fdef90c7bbdabf740a637e319 (patch) | |
| tree | 248d6b733e74bea2305f14b9706712d2c0d8e3ed /java/base.go | |
| parent | ca70fc40bd5d7bf3b24c25de0a06c2a6df17f7ea (diff) | |
Update max_sdk_version from SdkSpec to ApiLevel
max_sdk_version signifies device version and does not need an sdkKind to
describe it fully. Update the type and cleanup existing usages. As a
side benefit, we also get better error handling since users can no
longer enter something like `public_30` as a valid max_sdk_version in bp
files
Bug: 208456999
Test: no change in ninja file (this should be a no-op)
Test: TH
Change-Id: I304b5ad802bde200137d8e225182828dfd6f7227
Diffstat (limited to 'java/base.go')
| -rw-r--r-- | java/base.go | 22 |
1 files changed, 13 insertions, 9 deletions
diff --git a/java/base.go b/java/base.go index 2d213a192..1bcff2ed5 100644 --- a/java/base.go +++ b/java/base.go @@ -490,7 +490,7 @@ type Module struct { sdkVersion android.SdkSpec minSdkVersion android.ApiLevel - maxSdkVersion android.SdkSpec + maxSdkVersion android.ApiLevel sourceExtensions []string } @@ -672,16 +672,20 @@ func (j *Module) MinSdkVersion(ctx android.EarlyModuleContext) android.ApiLevel return j.SdkVersion(ctx).ApiLevel } -func (j *Module) MaxSdkVersion(ctx android.EarlyModuleContext) android.SdkSpec { - maxSdkVersion := proptools.StringDefault(j.deviceProperties.Max_sdk_version, "") - // SdkSpecFrom returns SdkSpecPrivate for this, which may be confusing. - // TODO(b/208456999): ideally MaxSdkVersion should be an ApiLevel and not SdkSpec. - return android.SdkSpecFrom(ctx, maxSdkVersion) +func (j *Module) MaxSdkVersion(ctx android.EarlyModuleContext) android.ApiLevel { + if j.deviceProperties.Max_sdk_version != nil { + return android.ApiLevelFrom(ctx, *j.deviceProperties.Max_sdk_version) + } + // Default is PrivateApiLevel + return android.SdkSpecPrivate.ApiLevel } -func (j *Module) ReplaceMaxSdkVersionPlaceholder(ctx android.EarlyModuleContext) android.SdkSpec { - replaceMaxSdkVersionPlaceholder := proptools.StringDefault(j.deviceProperties.Replace_max_sdk_version_placeholder, "") - return android.SdkSpecFrom(ctx, replaceMaxSdkVersionPlaceholder) +func (j *Module) ReplaceMaxSdkVersionPlaceholder(ctx android.EarlyModuleContext) android.ApiLevel { + if j.deviceProperties.Replace_max_sdk_version_placeholder != nil { + return android.ApiLevelFrom(ctx, *j.deviceProperties.Replace_max_sdk_version_placeholder) + } + // Default is PrivateApiLevel + return android.SdkSpecPrivate.ApiLevel } func (j *Module) MinSdkVersionString() string { |