only call jQuery(this) if necessary in $.fn.val

[jbedard]

This changes $.fn.val to only call jQuery(this) when it is used (when a function instead of value is passed to .val()).

Also saves a few bytes:
raw gz
-15 -8 dist/jquery.js
-4 -3 dist/jquery.min.js

[dmethvin]

@jbedard have you signed the CLA? The change itself looks good, always nice to make something smaller and faster.

[jbedard]

@dmethvin I've now signed the CLA.

[dmethvin]

It looks like this is now on a minor_changes branch? Do you want to submit all of those at once?

[jbedard]

I was going to submit those separately, or is one pull request with a few unrelated commits ok?

[dmethvin]

It just got a bit confusing since the branch has several unrelated commits. I've never seen a pull request that was essentially a cherry-pick of a branch, so it had me a bit confused. 😄 I guess I can pick off the commit from there, let me do that.

[mgol]

@jbedard yep, a good advice for the future - always create a separate branch for each pull request and submit the branch, not a single commit. It makes it easy to update pull request if there are any remarks from reviewers (and there often are), you just push a new commit to the branch and the pull request on GitHub is updated automatically.

[jbedard]

Thanks for the advice, I'll make sure I do that next time. I was intending on creating pull requests for the other changes I have in that branch, I guess I'll put those in their own branches first.

[jbedard]

I've put this change in it's own branch if you want to use that instead (https://github.com/jbedard/jquery/tree/val_no_jquery). Let me know if that would require a separate pull request.