Skip to content

Commit

Permalink
Not ambiguous after all.
Browse files Browse the repository at this point in the history
  • Loading branch information
johnno1962 committed Feb 21, 2024
1 parent bef73de commit 4169415
Showing 1 changed file with 1 addition and 11 deletions.
12 changes: 1 addition & 11 deletions proposals/0243-character-operators.md
Original file line number Diff line number Diff line change
Expand Up @@ -159,17 +159,7 @@ This last initializer is optional. It provides an alternate to the existing `Int

## Source compatibility

Our proposed operator suite is additive. After running the existing test suite, it does change diagnostics on a limited part of pattern matching code. We believe this diagnostic information was already flawed, and the change inconsequential. Finally, the last initializer (the one we noted as optional) may affect currently valid code, such as the following:

```
unicodeScalars.map(UInt32.init)
```

Upon adoption, this becomes ambiguous and will need to be rewritten explicitly as:

```
unicodeScalars.map { UInt32($0) }
```
Our proposed operator suite is additive. After running the existing test suite, it does change diagnostics on a limited part of invalid pattern matching code. We believe this diagnostic information was already flawed, and the change inconsequential.

## Effect on ABI stability

Expand Down

0 comments on commit 4169415

Please sign in to comment.