ded/qwery — comment on commit 65e225567052a8037d8cc1c6e2b6c211cadc9e02
File:  line 0
By rvagg:

This is all good except isn't strictly necessary because of the `try{}catch(){}` around the qSA calls which then defers to qwery() which in turn goes to the selectors.

There's even tests for it working:

* define strange custom pseudo here: https://github.com/ded/qwery/blob/65e225567052a8037d8cc1c6e2b6c211cadc9e02/tests/tests.js#L3
* invoke it here: https://github.com/ded/qwery/blob/65e225567052a8037d8cc1c6e2b6c211cadc9e02/tests/tests.js#L478-481

I believe also there's a slight performance hit from using `.apply`, particularly now in the newer engines because it's more difficult to optimise. So perhaps a `(qwery.nonStandardEngine ? selectNonNative : supportsCSS3 ? selectCSS3 : doc[qSA] ? selectCSS2qSA : selectNonNative)(selector, root)` (or whatever the arguments need to be)

I'm also a bit sad to see `.noConflict()` go. You're going to need to update the Bonzo tests: https://github.com/ded/bonzo/blob/7efcfb1fe51762daed1be574ab67f72b697caa42/tests/tests.html#L40