From 18f43cf20c8cd89439bdf27975868ce10d09b8a0 Mon Sep 17 00:00:00 2001 From: Chloe Stefantsova Date: Wed, 27 Nov 2024 09:44:45 +0000 Subject: [PATCH] [cfe] Remove obsolete TODOs Change-Id: I2e0d09850a8aeffdb1816cf81ae5f4595b1cfa02 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/397780 Reviewed-by: Johnni Winther Commit-Queue: Chloe Stefantsova --- pkg/kernel/lib/type_algebra.dart | 3 --- 1 file changed, 3 deletions(-) diff --git a/pkg/kernel/lib/type_algebra.dart b/pkg/kernel/lib/type_algebra.dart index 4faa805d3d4..c6bf27a8de5 100644 --- a/pkg/kernel/lib/type_algebra.dart +++ b/pkg/kernel/lib/type_algebra.dart @@ -1846,7 +1846,6 @@ class _PrimitiveTypeVerifier implements DartTypeVisitor { /// /// Some types are nullable even without the application of the nullable type /// constructor at the top level, for example, Null or FutureOr. -// TODO(cstefantsova): Remove [coreTypes] parameter when NullType is landed. DartType unwrapNullabilityConstructor(DartType type) { return type.accept(const _NullabilityConstructorUnwrapper()); } @@ -1855,8 +1854,6 @@ DartType unwrapNullabilityConstructor(DartType type) { /// /// Implementing the function as a visitor makes the necessity of supporting a /// new implementation of [DartType] visible at compile time. -// TODO(cstefantsova): Remove CoreTypes as the second argument when NullType is -// landed. class _NullabilityConstructorUnwrapper implements DartTypeVisitor { const _NullabilityConstructorUnwrapper();