Skip to content

Commit ee59ebd

Browse files
bcorsoDagger Team
authored andcommitted
Remove unnecessary workaround for b/242569252.
This bug was fixed in Kotlin 1.8, so it should be safe to remove this workaround. RELNOTES=N/A PiperOrigin-RevId: 806034576
1 parent 5426d64 commit ee59ebd

File tree

1 file changed

+0
-6
lines changed

1 file changed

+0
-6
lines changed

dagger-compiler/main/java/dagger/internal/codegen/base/DaggerSuperficialValidation.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -318,12 +318,6 @@ private void validateTypes(String desc, Collection<? extends XType> types) {
318318
*/
319319
private void validateType(String desc, XType type) {
320320
checkNotNull(type);
321-
// TODO(b/242569252): Due to a bug in kotlinc, a TypeName may incorrectly contain a "$" instead
322-
// of "." if the TypeName is requested before the type has been resolved. Furthermore,
323-
// XProcessing will cache the incorrect TypeName so that further calls will still contain the
324-
// "$" even after the type has been resolved. Thus, we try to resolve the type as early as
325-
// possible to prevent using/caching the incorrect TypeName.
326-
XTypes.resolveIfNeeded(type);
327321
try {
328322
if (isArray(type)) {
329323
validateType("array component type", asArray(type).getComponentType());

0 commit comments

Comments
 (0)