This repository was archived by the owner on Oct 17, 2024. It is now read-only.
Make _IdentityConverter compatible with Dart SDK 1.x #18
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Problem
#17 was merged and released in version
2.1.0
without changing the lower-bound of the Dart SDK to>=2.0.0
. The changes in #17 includes the construction of an_IdentityConverter
instance without thenew
orconst
keyword, which is a Dart SDK 2.x-only feature.This causes a regression for Dart 1.x SDK consumers building their application using the Dart Dev Compiler:
Solution
Use the
new
keyword to construct the instance. I attempted to keep it aconst
, but it does not appear thatconst
constructors can have generic parameters using the Dart 1.x SDK.@jonasfj @kevmoo @lrhn @jayudey-wf @robbecker-wf