Skip to content

Fix compiler warnings in ByteCodeGen, Compiler, and Regex#854

Merged
tshortli merged 1 commit intoswiftlang:mainfrom
tshortli:fix-warnings
Feb 24, 2026
Merged

Fix compiler warnings in ByteCodeGen, Compiler, and Regex#854
tshortli merged 1 commit intoswiftlang:mainfrom
tshortli:fix-warnings

Conversation

@tshortli
Copy link
Copy Markdown
Contributor

  • Replace unused let bindings with _ in ByteCodeGen+DSLList.swift
  • Replace unused var codegen with _ = in Compiler.swift
  • Add @unknown default to exhaustive switches on AST.Node in ASTConversion.swift since the switch enumerates cases from a library with library evolution in some build configurations
  • Fix unused bindings and no-effect let patterns in DSLTree.swift and ASTConversion.swift

- Replace unused `let` bindings with `_` in ByteCodeGen+DSLList.swift
- Replace unused `var codegen` with `_ =` in Compiler.swift
- Add `@unknown default` to exhaustive switches on `AST.Node` in ASTConversion.swift
- Fix unused bindings and no-effect `let` patterns in DSLTree.swift and ASTConversion.swift
@tshortli
Copy link
Copy Markdown
Contributor Author

@swift-ci please test

@tshortli
Copy link
Copy Markdown
Contributor Author

@natecook1000 I'm planning to merge this shortly but I'm happy to address feedback in a follow up if you have any

@tshortli tshortli merged commit 6a693bb into swiftlang:main Feb 24, 2026
22 checks passed
@tshortli tshortli deleted the fix-warnings branch February 24, 2026 18:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant