Skip to content

Commit 2d2901b

Browse files
bwilkersonCommit Queue
authored andcommitted
Fix a bug in since_sdk_version.dart
Discovered while attempting to migrate the file, split out for easier review. Change-Id: I1a3c56e0c277c79e9e71db89c8d5c8aa26891cc7 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/403283 Reviewed-by: Konstantin Shcheglov <[email protected]> Commit-Queue: Brian Wilkerson <[email protected]>
1 parent 701bf9c commit 2d2901b

File tree

2 files changed

+10
-1
lines changed

2 files changed

+10
-1
lines changed

pkg/analyzer/lib/src/dart/element/since_sdk_version.dart

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ class SinceSdkVersionComputer {
6060
}
6161
}
6262
}
63-
return null;
63+
return result;
6464
}
6565

6666
/// Returns the parsed [Version], or `null` if wrong format.

pkg/analyzer/test/src/summary/elements/since_sdk_version_test.dart

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -384,6 +384,7 @@ library
384384
foo
385385
reference: dart:foo::@class::A::@method::foo
386386
firstFragment: dart:foo::<fragment>::@class::A::@method::foo
387+
sinceSdkVersion: 2.16.0
387388
''');
388389
}
389390

@@ -440,6 +441,7 @@ library
440441
foo
441442
reference: dart:foo::@class::A::@method::foo
442443
firstFragment: dart:foo::<fragment>::@class::A::@method::foo
444+
sinceSdkVersion: 2.14.0
443445
''');
444446
}
445447

@@ -906,6 +908,7 @@ library
906908
foo
907909
reference: dart:foo::@function::foo
908910
firstFragment: dart:foo::<fragment>::@function::foo
911+
sinceSdkVersion: 2.15.0
909912
returnType: void
910913
bar
911914
reference: dart:foo::@function::bar
@@ -950,6 +953,7 @@ library
950953
foo
951954
reference: dart:foo::@function::foo
952955
firstFragment: dart:foo::<fragment>::@function::foo
956+
sinceSdkVersion: 2.15.3-dev.7
953957
returnType: void
954958
''');
955959
}
@@ -990,6 +994,7 @@ library
990994
foo
991995
reference: dart:foo::@function::foo
992996
firstFragment: dart:foo::<fragment>::@function::foo
997+
sinceSdkVersion: 2.15.3
993998
returnType: void
994999
''');
9951000
}
@@ -1132,6 +1137,7 @@ library
11321137
optionalNamed p2
11331138
firstFragment: dart:foo::<fragment>::@function::f::@parameter::p2
11341139
type: int?
1140+
sinceSdkVersion: 2.15.0
11351141
returnType: void
11361142
''');
11371143
}
@@ -1189,6 +1195,7 @@ library
11891195
type: int
11901196
optionalPositional p2
11911197
type: int?
1198+
sinceSdkVersion: 2.15.0
11921199
returnType: void
11931200
''');
11941201
}
@@ -1227,6 +1234,7 @@ library
12271234
typeAliases
12281235
A
12291236
firstFragment: dart:foo::<fragment>::@typeAlias::A
1237+
sinceSdkVersion: 2.15.0
12301238
aliasedType: List<int>
12311239
''');
12321240
}
@@ -1280,6 +1288,7 @@ library
12801288
final hasInitializer foo
12811289
reference: dart:foo::@topLevelVariable::foo
12821290
firstFragment: dart:foo::<fragment>::@topLevelVariable::foo
1291+
sinceSdkVersion: 2.15.0
12831292
type: int
12841293
getter: dart:foo::<fragment>::@getter::foo#element
12851294
getters

0 commit comments

Comments
 (0)