Skip to content

Commit

Permalink
Flow analysis: fix capitalization convention.
Browse files Browse the repository at this point in the history
Both letters of `OK` should be capitalized, in accordance with
https://dart.dev/effective-dart/style#do-capitalize-acronyms-and-abbreviations-longer-than-two-letters-like-words.

Change-Id: I920436d373d1a597244e5592e568a19d0f22fa86
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/332065
Reviewed-by: Lasse Nielsen <[email protected]>
Commit-Queue: Paul Berry <[email protected]>
  • Loading branch information
stereotype441 authored and Commit Queue committed Oct 27, 2023
1 parent d42390d commit e7c88f1
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
4 changes: 2 additions & 2 deletions pkg/_fe_analyzer_shared/test/mini_ast.dart
Original file line number Diff line number Diff line change
Expand Up @@ -1804,7 +1804,7 @@ class Harness {
/// Runs the given [statements] through flow analysis, checking any assertions
/// they contain.
void run(List<ProtoStatement> statements,
{bool errorRecoveryOk = false, Set<String> expectedErrors = const {}}) {
{bool errorRecoveryOK = false, Set<String> expectedErrors = const {}}) {
try {
_started = true;
if (_operations.legacy && patternsEnabled) {
Expand All @@ -1826,7 +1826,7 @@ class Harness {
expect(typeAnalyzer.errors._accumulatedErrors, expectedErrors);
var assertInErrorRecoveryStack =
typeAnalyzer.errors._assertInErrorRecoveryStack;
if (!errorRecoveryOk && assertInErrorRecoveryStack != null) {
if (!errorRecoveryOK && assertInErrorRecoveryStack != null) {
fail('assertInErrorRecovery called but no errors reported: '
'$assertInErrorRecoveryStack');
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1473,7 +1473,7 @@ main() {
],
).checkIR('switch(expr(int), case(heads(variables()), '
'block(break())))'),
], errorRecoveryOk: true);
], errorRecoveryOK: true);
});

test('Multiple', () {
Expand Down

0 comments on commit e7c88f1

Please sign in to comment.