Side navigation
#9878 closed bug (duplicate)
Opened July 21, 2011 12:50AM UTC
Closed July 21, 2011 10:41AM UTC
Last modified July 21, 2011 10:41AM UTC
IE needs catch() before finally
| Reported by: | scraze@hoekje.org | 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!)
Attachments (0)
Change History (5)
Changed July 21, 2011 02:14AM UTC by comment:1
| owner: | → scraze@hoekje.org |
|---|---|
| status: | new → pending |
Changed July 21, 2011 02:44AM UTC by comment:2
| status: | pending → new |
|---|
Replying to [comment:1 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.
Changed July 21, 2011 04:30AM UTC by comment:3
| component: | unfiled → ajax |
|---|---|
| owner: | scraze@hoekje.org → jaubourg |
| priority: | undecided → high |
| status: | new → assigned |
Thanks!
Changed July 21, 2011 10:41AM UTC by comment:4
| resolution: | → duplicate |
|---|---|
| status: | assigned → closed |
Are you actually referring to this:
https://github.com/jquery/jquery/blob/master/src/deferred.js#L61-62