Skip to content
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
merged 1 commit into from
Dec 3, 2015
Merged

Fixing wrong method documentation for SetAlgebra #95

merged 1 commit into from
Dec 3, 2015

Conversation

regexident
Copy link
Contributor

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

😉

I had reported a bug on this on Aug 4th 2015 (rdar://22031133).
Radar screening staff however deemed it not even worthy reading, apparently.

This issue behaves as intended based on the following:
Your suggested rewrite is isn’t accurate—‘subtract’ doesn’t return a Boolean at all.
We are now closing this bug report.

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!

…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:
@regexident regexident changed the title Fixing wrong method documentation for SetAlgebra (see my rdar://22031133) Fixing wrong method documentation for SetAlgebra Dec 3, 2015
bitjammer added a commit that referenced this pull request Dec 3, 2015
Fixing wrong method documentation for SetAlgebra.subtract
@bitjammer bitjammer merged commit ae24c99 into swiftlang:master Dec 3, 2015
@regexident
Copy link
Contributor Author

Damn! Color me impressed. 😮

Man, I'm already in love with this Apple! 😍 🙇

@regexident regexident deleted the SetAlgebra-docfix branch December 3, 2015 21:14
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
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants