Skip to content

Commit f6d66b6

Browse files
srawlinsCommit Queue
authored and
Commit Queue
committed
Update fix status file
Change-Id: I9ac3d082720f9176704f1c600d96238f55778a63 Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/271240 Reviewed-by: Brian Wilkerson <[email protected]> Commit-Queue: Samuel Rawlins <[email protected]>
1 parent adc8543 commit f6d66b6

File tree

1 file changed

+34
-14
lines changed

1 file changed

+34
-14
lines changed

pkg/analysis_server/lib/src/services/correction/error_fix_status.yaml

Lines changed: 34 additions & 14 deletions
Original file line numberDiff line numberDiff line change
@@ -24,11 +24,12 @@
2424
# issue created for it.
2525
#
2626
# Stats:
27-
# - 717 "needsEvaluation"
28-
# - 220 for ParserErrorCodes.
29-
# - 72 "needsFix"
30-
# - 296 "hasFix"
31-
# - 57 "noFix"
27+
# - 710 "needsEvaluation"
28+
# - 282 for CompileTimeErrorCodes
29+
# - 220 for ParserErrorCodes
30+
# - 70 "needsFix"
31+
# - 298 "hasFix"
32+
# - 64 "noFix"
3233

3334
AnalysisOptionsErrorCode.INCLUDED_FILE_PARSE_ERROR:
3435
status: noFix
@@ -146,19 +147,34 @@ CompileTimeErrorCode.BODY_MIGHT_COMPLETE_NORMALLY:
146147
CompileTimeErrorCode.BREAK_LABEL_ON_SWITCH_MEMBER:
147148
status: needsEvaluation
148149
CompileTimeErrorCode.BUILT_IN_IDENTIFIER_AS_EXTENSION_NAME:
149-
status: needsEvaluation
150+
status: noFix
151+
notes: |-
152+
The correction is to change the name.
150153
CompileTimeErrorCode.BUILT_IN_IDENTIFIER_AS_PREFIX_NAME:
151-
status: needsEvaluation
154+
status: noFix
155+
notes: |-
156+
The correction is to change the name.
152157
CompileTimeErrorCode.BUILT_IN_IDENTIFIER_AS_TYPE:
153-
status: needsEvaluation
158+
status: noFix
159+
notes: |-
160+
The correction is to change the name.
154161
CompileTimeErrorCode.BUILT_IN_IDENTIFIER_AS_TYPE_NAME:
155-
status: needsEvaluation
162+
status: noFix
163+
notes: |-
164+
The correction is to change the name.
156165
CompileTimeErrorCode.BUILT_IN_IDENTIFIER_AS_TYPE_PARAMETER_NAME:
157-
status: needsEvaluation
166+
status: noFix
167+
notes: |-
168+
The correction is to change the name.
158169
CompileTimeErrorCode.BUILT_IN_IDENTIFIER_AS_TYPEDEF_NAME:
159-
status: needsEvaluation
170+
status: noFix
171+
notes: |-
172+
The correction is to change the name.
160173
CompileTimeErrorCode.CASE_BLOCK_NOT_TERMINATED:
161-
status: needsEvaluation
174+
status: needsFix
175+
notes: |-
176+
The typical fix, when someone thought a termination was not needed, is to
177+
add a break.
162178
CompileTimeErrorCode.CASE_EXPRESSION_TYPE_IMPLEMENTS_EQUALS:
163179
status: needsEvaluation
164180
CompileTimeErrorCode.CASE_EXPRESSION_TYPE_IS_NOT_SWITCH_EXPRESSION_SUBTYPE:
@@ -363,9 +379,13 @@ CompileTimeErrorCode.ENUM_CONSTANT_SAME_NAME_AS_ENCLOSING:
363379
notes: |-
364380
The fix is to rename one of the two, but we can't know what name to use.
365381
CompileTimeErrorCode.ENUM_CONSTANT_WITH_NON_CONST_CONSTRUCTOR:
366-
status: needsFix
382+
status: noFix
367383
since: 2.17
368-
notes: Use AddConst.new
384+
notes: |-
385+
This is only reported on enum constant declarations calling a factory
386+
constructor. No enum factory constructor can be const. The correction is to
387+
call a different constructor, or refactor the factory constructor to be
388+
const and generative.
369389
CompileTimeErrorCode.ENUM_INSTANTIATED_TO_BOUNDS_IS_NOT_WELL_BOUNDED:
370390
status: noFix
371391
since: 2.17

0 commit comments

Comments
 (0)