Skip to content
This repository has been archived by the owner on Dec 10, 2024. It is now read-only.

should methods have @ discardableResult attribute? #42

Closed
alcamla opened this issue Sep 12, 2016 · 2 comments
Closed

should methods have @ discardableResult attribute? #42

alcamla opened this issue Sep 12, 2016 · 2 comments

Comments

@alcamla
Copy link

alcamla commented Sep 12, 2016 •
edited
Loading

In the cases where the operations (put, post, etc) are performed asynchronously, the result is ignored and the manipulation of the result is done in the completion handler. Currently this warning must be silenced manually, but I wonder if the @ discardableResult could be used instead.
@dduan
Copy link
Owner

dduan commented Sep 12, 2016

Ah, great catch. Thank you!

@dduan
Copy link
Owner

dduan commented Sep 13, 2016

Done in release 0.5.1.

@dduan dduan closed this as completed Sep 13, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dduan @alcamla