Modify ↓
Ticket #10947 (closed bug: invalid)
Remove redundant return value in $.Callbacks
| Reported by: | dcherman | Owned by: | |
|---|---|---|---|
| Priority: | undecided | Milestone: | None |
| Component: | unfiled | Version: | git |
| Keywords: | Cc: | ||
| Blocking: | Blocked by: |
Description
https://github.com/jquery/jquery/blob/master/src/callbacks.js#L213-215
fireWith already returns this, so you can save a couple of bytes by changing fire to just return self.fireWith( this, arguments );
Change History
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.
Note: See
TracTickets for help on using
tickets.

Ugh never mind, close this as invalid please.
Should have tested this locally first, it breaks the Deferred chainability tests.