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

Browse Source
This commit is contained in:
Michael Wright
2024-04-26 12:44:37 +00:00
committed by Gerrit Code Review
parent 68d63d47af 04e4564f49
commit d71cd85c0a
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 // ktfmt doesn't understand the `..<` range syntax; explicitly call .rangeUntil instead
for (i in 0.rangeUntil(fields.getLength())) { for (i in 0.rangeUntil(fields.getLength())) {
val field = fields.item(i) 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 = val className =
requireNotNull(field.getParentNode()) { "Bad XML: top level <field> element" } requireNotNull(field.getParentNode()) { "Bad XML: top level <field> element" }
.getAttribute("name") .getAttribute("name")