Skip to content

stdlib: Remove unused, unsafe helper function _withUninitializedString #33704

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 14, 2020

Conversation

belkadan
Copy link
Contributor

No functionality change.

@benrimmington
Copy link
Contributor

@swift-ci Please smoke test

@benrimmington
Copy link
Contributor

@swift-ci Please smoke test macOS platform

@MaxDesiatov
Copy link
Contributor

Does anyone mind if this is merged now?

@belkadan
Copy link
Contributor Author

Maybe people were expecting me to merge it, but my personal account doesn't have commit access. (And I don't use my Apple account anymore.)

@MaxDesiatov MaxDesiatov merged commit fc6b14f into swiftlang:master Sep 14, 2020
@belkadan belkadan deleted the long-dead-code-elimination branch September 15, 2020 18:35
dabrahams pushed a commit that referenced this pull request Sep 15, 2020
* 'master' of github.com:apple/swift:
  [docs] Fix broken links in the documentation index (#33829)
  [Profiler] Increment closure body count prior to the prolog (#33946)
  ARCSequenceOpts: Add LoopSummary verifier (#33810)
  [build-script] Cleanup source directory layout help
  Address review comment.
  ABI checker: when invoking via build system, explicitly mention ABI breakge in diagnostics
  stdlib: Remove unused, unsafe helper function _withUninitializedString (#33704)
  test: Replace _silgen_name w/ _cdecl in CommandLineStressTest
  [AutoDiff] [Docs] Clarify 'Differentiable' derived conformances conditions.
  Revert "build-script: remove dead CMake options for Swift"
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.

4 participants