Most try/catch are unneeded #1

Closed
rauchg opened this Issue Nov 23, 2012 · 3 comments

Projects

None yet

2 participants

@yields yields pushed a commit that closed this issue Nov 23, 2012
Amir Abu Shareb remove unneeded try's. closes #1 1cbb335
@yields yields closed this in 1cbb335 Nov 23, 2012
@yields
Owner
yields commented Nov 23, 2012

Microsoft is confused just as much as we are....
http://msdn.microsoft.com/en-us/library/ms537505(v=vs.85).aspx

I don't know who to believe any more haha..

@rauchg
rauchg commented Nov 24, 2012

Haha probably going with [https://github.com/jquery/jquery/blob/master/src/ajax/xhr.js#L20](what jQuery uses) would be pretty safe:

function createActiveXHR() {
    try {
        return new window.ActiveXObject( "Microsoft.XMLHTTP" );
    } catch( e ) {}
}
@yields
Owner
yields commented Nov 24, 2012

@guille Yep i saw that as well, but in the article you gave me it said that Microsoft. is an older namespace, then
in wiki they have Microsoft.XMLHTTP but below it there is a comment
that states: //Microsoft.XMLHTTP points to Msxml2.XMLHTTP and is redundant haha.

The whole thing is confusing but it works in ie6, ie7.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment