#9878 closed bug (duplicate)
IE needs catch() before finally
| Reported by: | Owned by: | jaubourg | |
|---|---|---|---|
| Priority: | high | Milestone: | None |
| Component: | ajax | Version: | 1.6.1 |
| Keywords: | Cc: | ||
| Blocked by: | Blocking: |
Description
Tested with IE8.0 on Windows Vista Home Premium: IE gives a warning ("Invalid argument") when it runs into 'finally' on line 998, in the try starting on line 993. According to random google result "http://webbugtrack.blogspot.com/2007/11/bug-184-catch-to-try-catch-finally-in.html", finally isn't even executed without catch. Either way, an empty catch such as on line 4894 ("catch(pseudoError) {}") resolves the issue. (Thanks!)
Change History (5)
comment:1 follow-up: 2 Changed 9 years ago by
| Owner: | set to scraze@… |
|---|---|
| Status: | new → pending |
comment:2 Changed 9 years ago by
| Status: | pending → new |
|---|
Replying to rwaldron:
Are you actually referring to this:
https://github.com/jquery/jquery/blob/master/src/deferred.js#L61-62
Indeed I'm referring to the highlighted lines - I failed to mention it's in the function resolveWith.
comment:3 Changed 9 years ago by
| Component: | unfiled → ajax |
|---|---|
| Owner: | changed from scraze@… to jaubourg |
| Priority: | undecided → high |
| Status: | new → assigned |
Thanks!
comment:4 Changed 9 years ago by
| Resolution: | → duplicate |
|---|---|
| Status: | assigned → closed |
Note: See
TracTickets for help on using
tickets.
Are you actually referring to this:
https://github.com/jquery/jquery/blob/master/src/deferred.js#L61-62