Skip to content

Commit

Permalink
[Diagnostics-Qol]: update c_style_for_stmt_removed error string
Browse files Browse the repository at this point in the history
  • Loading branch information
saehejkang committed Apr 28, 2024
1 parent 49dd95f commit 2835376
Show file tree
Hide file tree
Showing 6 changed files with 19 additions and 19 deletions.
2 changes: 1 addition & 1 deletion include/swift/AST/DiagnosticsParse.def
Original file line number Diff line number Diff line change
Expand Up @@ -1189,7 +1189,7 @@ ERROR(do_throws_without_catch,none,

// C-Style For Stmt
ERROR(c_style_for_stmt_removed,none,
"C-style for statement has been removed in Swift 3", ())
"C-style for statement was removed in Swift 3", ())

// For-each Stmt
ERROR(expected_foreach_in,PointsToFirstBadToken,
Expand Down
8 changes: 4 additions & 4 deletions test/Parse/recovery.swift
Original file line number Diff line number Diff line change
Expand Up @@ -145,17 +145,17 @@ func missingWhileInRepeat() {
func acceptsClosure<T>(t: T) -> Bool { return true }

func missingControllingExprInFor() {
for ; { // expected-error {{C-style for statement has been removed in Swift 3}}
for ; { // expected-error {{C-style for statement was removed in Swift 3}}
}

for ; // expected-error {{C-style for statement has been removed in Swift 3}}
for ; // expected-error {{C-style for statement was removed in Swift 3}}
{
}

for ; true { // expected-error {{C-style for statement has been removed in Swift 3}}
for ; true { // expected-error {{C-style for statement was removed in Swift 3}}
}

for var i = 0; true { // expected-error {{C-style for statement has been removed in Swift 3}}
for var i = 0; true { // expected-error {{C-style for statement was removed in Swift 3}}
i += 1
}
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1685,7 +1685,7 @@
key.column: 1,
key.severity: source.diagnostic.severity.error,
key.id: "c_style_for_stmt_removed",
key.description: "C-style for statement has been removed in Swift 3",
key.description: "C-style for statement was removed in Swift 3",
key.diagnostic_stage: source.diagnostic.stage.swift.parse,
key.ranges: [
{
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -1687,7 +1687,7 @@
key.filepath: "-foobar",
key.severity: source.diagnostic.severity.error,
key.id: "c_style_for_stmt_removed",
key.description: "C-style for statement has been removed in Swift 3",
key.description: "C-style for statement was removed in Swift 3",
key.diagnostic_stage: source.diagnostic.stage.swift.parse,
key.ranges: [
{
Expand Down
2 changes: 1 addition & 1 deletion test/SourceKit/DocumentStructure/structure.swift.response
Original file line number Diff line number Diff line change
Expand Up @@ -1687,7 +1687,7 @@
key.filepath: main.swift,
key.severity: source.diagnostic.severity.error,
key.id: "c_style_for_stmt_removed",
key.description: "C-style for statement has been removed in Swift 3",
key.description: "C-style for statement was removed in Swift 3",
key.diagnostic_stage: source.diagnostic.stage.swift.parse,
key.ranges: [
{
Expand Down
22 changes: 11 additions & 11 deletions test/stmt/c_style_for.swift
Original file line number Diff line number Diff line change
@@ -1,40 +1,40 @@
// RUN: %target-typecheck-verify-swift

for var i = 0; i < 10; i++ {}
// expected-error @-1 {{C-style for statement has been removed in Swift 3}} {{none}}
// expected-error @-1 {{C-style for statement was removed in Swift 3}} {{none}}

for var i = 0; i < 10; i += 1 {}
// expected-error @-1 {{C-style for statement has been removed in Swift 3}} {{none}}
// expected-error @-1 {{C-style for statement was removed in Swift 3}} {{none}}

for var i = 0; i <= 10; i++ {}
// expected-error @-1 {{C-style for statement has been removed in Swift 3}} {{none}}
// expected-error @-1 {{C-style for statement was removed in Swift 3}} {{none}}

for var i = 0; i <= 10; i += 1 {}
// expected-error @-1 {{C-style for statement has been removed in Swift 3}} {{none}}
// expected-error @-1 {{C-style for statement was removed in Swift 3}} {{none}}

for var i = 10; i > 0; i-- {}
// expected-error @-1 {{C-style for statement has been removed in Swift 3}} {{none}}
// expected-error @-1 {{C-style for statement was removed in Swift 3}} {{none}}

for var i = 10; i > 0; i -= 1 {}
// expected-error @-1 {{C-style for statement has been removed in Swift 3}} {{none}}
// expected-error @-1 {{C-style for statement was removed in Swift 3}} {{none}}

for var i = 10; i >= 0; i-- {}
// expected-error @-1 {{C-style for statement has been removed in Swift 3}} {{none}}
// expected-error @-1 {{C-style for statement was removed in Swift 3}} {{none}}

for var i = 10; i >= 0; i -= 1 {}
// expected-error @-1 {{C-style for statement has been removed in Swift 3}} {{none}}
// expected-error @-1 {{C-style for statement was removed in Swift 3}} {{none}}

let start = Int8(4)
let count = Int8(10)
var other = Int8(2)

for ; other<count; other+=1 { // expected-error {{C-style for statement has been removed in Swift 3}} {{none}}
for ; other<count; other+=1 { // expected-error {{C-style for statement was removed in Swift 3}} {{none}}
}

for (var number : Int8 = start; number < count; number+=1) { // expected-error {{C-style for statement has been removed in Swift 3}} {{none}}
for (var number : Int8 = start; number < count; number+=1) { // expected-error {{C-style for statement was removed in Swift 3}} {{none}}
print(number)
}

for (var m : Int8 = start; m < count; m+=1) { // expected-error {{C-style for statement has been removed in Swift 3}} {{none}}
for (var m : Int8 = start; m < count; m+=1) { // expected-error {{C-style for statement was removed in Swift 3}} {{none}}
m += 3
}

0 comments on commit 2835376

Please sign in to comment.