Fixed
Status Update
Comments
ch...@google.com <ch...@google.com> #2
Can you use the workaround described at
kf...@gmail.com <kf...@gmail.com> #3
Using 8.4.40 or 8.5.26 cannot fix the issue.
8.3.51 is not found.
Using 8.3.37 from
kf...@gmail.com <kf...@gmail.com> #4
From
@Søren: Could you triage this issue?
an...@google.com <an...@google.com>
an...@google.com <an...@google.com>
an...@google.com <an...@google.com>
an...@google.com <an...@google.com> #5
Thanks for taking the time to create a reproduction. I reproduced the issue and confirmed that the issue is still present on main (R8 8.6). I will land a fix soon and backport the fix to the relevant release branches.
an...@google.com <an...@google.com> #6
Project: r8
Branch: main
commit 7803ec93afef8bf1047eaf8f209fdbe81bdbd581
Author: Christoffer Adamsen <christofferqa@google.com>
Date: Fri Jun 28 08:39:10 2024
Reproduce IAE from instance field merging
Bug: b/347676160
Change-Id: I0614da65d0a60f7f2d01d6a8aa92b9cadfabe005
A src/test/java/com/android/tools/r8/classmerging/horizontal/InaccessibleFieldTypeMergingTest.java
A src/test/java/com/android/tools/r8/classmerging/horizontal/testclasses/InaccessibleFieldTypeMergingTestClasses.java
https://r8-review.googlesource.com/94140
Branch: main
commit 7803ec93afef8bf1047eaf8f209fdbe81bdbd581
Author: Christoffer Adamsen <christofferqa@google.com>
Date: Fri Jun 28 08:39:10 2024
Reproduce IAE from instance field merging
Bug:
Change-Id: I0614da65d0a60f7f2d01d6a8aa92b9cadfabe005
A src/test/java/com/android/tools/r8/classmerging/horizontal/InaccessibleFieldTypeMergingTest.java
A src/test/java/com/android/tools/r8/classmerging/horizontal/testclasses/InaccessibleFieldTypeMergingTestClasses.java
kf...@gmail.com <kf...@gmail.com> #7
Project: r8
Branch: main
commit 9ea42409d593d89dce0184f00728ccd46ba5e7a3
Author: Christoffer Adamsen <christofferqa@google.com>
Date: Fri Jun 28 12:48:42 2024
Disallow class merging when instance field merging may lead to IAE
Bug: b/347676160
Change-Id: I521dc342150e8374992dd453379915e8a5e36db8
M src/main/java/com/android/tools/r8/shaking/Enqueuer.java
M src/test/java/com/android/tools/r8/classmerging/horizontal/InaccessibleFieldTypeMergingTest.java
https://r8-review.googlesource.com/94142
Branch: main
commit 9ea42409d593d89dce0184f00728ccd46ba5e7a3
Author: Christoffer Adamsen <christofferqa@google.com>
Date: Fri Jun 28 12:48:42 2024
Disallow class merging when instance field merging may lead to IAE
Bug:
Change-Id: I521dc342150e8374992dd453379915e8a5e36db8
M src/main/java/com/android/tools/r8/shaking/Enqueuer.java
M src/test/java/com/android/tools/r8/classmerging/horizontal/InaccessibleFieldTypeMergingTest.java
Description
To prevent users of your library from accessing resources intended only for internal use, you can make all resources private by adding an empty <public /> tag. This marks nothing as public and makes all resources private.
However, Android Studio shows errors stating that 'name' and 'type' attributes should be defined.
STEPS TO REPRODUCE:
1. Create a library module.
2. Create the public.xml file in the res/values/ directory.
3. Add an empty <public /> tag.
ATTACH SCREENSHOTS/RECORDINGS OF THE ISSUE
ATTACH LOG FILES (Select Help > Show Log in Files, or Show Log in Finder on a Mac)
------------------
IMPORTANT: Please read
all required information.
------------------
Studio Build: Build #AI-233.14808.21.2331.11842104, built on May 15, 2024
Version of Gradle Plugin: 8.4.1
Version of Gradle: 8.6
Version of Java: 17.0.10+0-17.0.10b1087.21-11572160 aarch64
OS: macOS 14.5 (23F79)