fix isPackageId and isResourceId

Tahgolov · Tahgolov · commit 4a5ec24a38ba · 2025-08-17T20:54:23.000+03:00
diff --git a/src/main/java/com/reandroid/arsc/chunk/PackageBlock.java b/src/main/java/com/reandroid/arsc/chunk/PackageBlock.java
@@ -811,10 +811,10 @@ public String toString(){
         return builder.toString();
     }
     public static boolean isPackageId(int packageId){
-        return (byte)packageId > 0;
+        return packageId > 0 && packageId <= 0xff;
     }
     public static boolean isResourceId(int resourceId){
-        return (resourceId >>> 16) != 0;
+        return ((resourceId >> 24) & (byte) (resourceId >> 16)) != 0;
     }
 
     public static void changePackageId(ValueItem valueItem, int packageIdOld, int packageIdNew){

Original file line number	Diff line number	Diff line change
`@@ -811,10 +811,10 @@ public String toString(){`
`811`	`811`	`return builder.toString();`
`812`	`812`	`}`
`813`	`813`	`public static boolean isPackageId(int packageId){`
`814`		`- return (byte)packageId > 0;`
	`814`	`+ return packageId > 0 && packageId <= 0xff;`
`815`	`815`	`}`
`816`	`816`	`public static boolean isResourceId(int resourceId){`
`817`		`- return (resourceId >>> 16) != 0;`
	`817`	`+ return ((resourceId >> 24) & (byte) (resourceId >> 16)) != 0;`
`818`	`818`	`}`
`819`	`819`
`820`	`820`	`public static void changePackageId(ValueItem valueItem, int packageIdOld, int packageIdNew){`