-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
Fixing wrong method documentation for SetAlgebra #95
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…133) Spot the error: /// Return true iff `self.intersect(other).isEmpty`. @warn_unused_result func subtract(other: Self) -> Self … /// Return true iff `self.intersect(other).isEmpty`. @warn_unused_result func isDisjointWith(other: Self) -> Bool :wink:
bitjammer
added a commit
that referenced
this pull request
Dec 3, 2015
Fixing wrong method documentation for SetAlgebra.subtract
Damn! Color me impressed. 😮 Man, I'm already in love with this Apple! 😍 🙇 |
gottesmm
added a commit
to gottesmm/swift
that referenced
this pull request
Sep 28, 2016
…ed out hashes for each repo. I think update-checkout is growing to the point, we should probably rename it to something like repo-tool. The output of this command looks as follows: clang 973bd1a Merge remote-tracking branch 'origin/swift-3.1-branch' into stable cmark 5af77f3 Merge pull request swiftlang#95 from kainjow/master compiler-rt 1f24bd0 Merge remote-tracking branch 'origin/swift-3.1-branch' into stable llbuild c324ee3 Merge pull request swiftlang#35 from tinysun212/pr-cygwin-1 lldb f6a5830 Adjust LLDB for changes to the layout of _SwiftTypePreservingNSNumber llvm 52482d0 Merge remote-tracking branch 'origin/swift-3.1-branch' into stable swift 45f3d2a [update-checkout] Add a small tool to dump hashes for all of the checkout repos. swift-corelibs-foundation cc5985e Loopback tests for URLSession (swiftlang#613) swift-corelibs-libdispatch ba7802e Merge pull request swiftlang#178 from dgrove-oss/depend-on-swiftc swift-corelibs-xctest 51b419d Merge pull request swiftlang#174 from modocache/sr-1901-remove-workarounds swift-integration-tests c95c832 Merge pull request swiftlang#12 from abertelrud/fix-swift-package-init-lib-test swift-xcode-playground-support 4b40c34 Merge pull request swiftlang#10 from apple/stdlib-unittest-expect-nil swiftpm 65403f5 [ConventionTests] Collect all the diagnostics from PackageBuilder
dabelknap
pushed a commit
to dabelknap/swift
that referenced
this pull request
Aug 2, 2018
dabelknap
pushed a commit
to dabelknap/swift
that referenced
this pull request
Sep 17, 2018
maldahleh
pushed a commit
to maldahleh/swift
that referenced
this pull request
Oct 26, 2020
Doc: clarify building SwiftPM dependency doesn't require the compiler source.
freak4pc
pushed a commit
to freak4pc/swift
that referenced
this pull request
Sep 28, 2022
…n-linux [common] Make call to chflags platform-conditional on Darwin
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Spot the error:
😉
I had reported a bug on this on Aug 4th 2015 (rdar://22031133).
Radar screening staff however deemed it not even worthy reading, apparently.
It does indeed not return a boolean. That was the whole point of my radar. Sigh.
So here I am, fixing the damn thing myself. 😎
Ps: Thanks @lattner & Team! You made my day with this release! 🙇
Your attitude of friendliness and openness really makes a difference at Apple!