StrixOS/build
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "check-flagged-apis: add missing requireNotNull" into main am: d71cd85c0a

Browse Source 
Original change: https://android-review.googlesource.com/c/platform/build/+/3063764

Change-Id: I6d3e6527e8193830b977dcb0def77c8d95455f32
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Michael Wright
2024-04-26 13:02:25 +00:00
committed by Automerger Merge Worker
parent b59a63495f d71cd85c0a
commit 8a25575172
1 changed files with 4 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
tools/check-flagged-apis/src/com/android/checkflaggedapis/Main.kt
View File

@@ -207,7 +207,10 @@ internal fun parseApiVersions(input: InputStream): Set<Symbol> {
// ktfmt doesn't understand the `..<` range syntax; explicitly call .rangeUntil instead
for (i in 0.rangeUntil(fields.getLength())) {
val field = fields.item(i)
val fieldName = field.getAttribute("name")
val fieldName =
requireNotNull(field.getAttribute("name")) {
"Bad XML: <field> element without name attribute"
}
val className =
requireNotNull(field.getParentNode()) { "Bad XML: top level <field> element" }
.getAttribute("name")