Tried all these cases in IE6, FF2, Opera 9.25 and Safari 3 and they all worked well.

var keys = $.map( {a:1,b:2}, function( v, k ){
	return k;
}, [ ] );
alert(keys);

var values = $.map( {a:1,b:2}, function( v, k ){
	return v;
}, [ ] );
alert(values);

var mapped = $.map( document.getElementsByTagName('script'), function( v, k ){
	return v;
}, {length:0} );
alert(mapped);

var flat = $.map( Array(8), function( v, k ){
	return k % 2 ? k : [k,k,k];//try mixing array and regular returns
});
alert(flat);

Could you convert that to the testsuite, too?

Commited test in [5342].

Your patch must be tested against the benchmark before applying it, as map is a core function of jQuery. Its raw speed has higher priority then being useful for other cases. If we can do both, even better.

Shouldn't you/we accept the change before adding the tests to the test suite ?

Or it will fail for others that haven't applied the proposed diff.

I meant you/we add the test, not accept.

I didn't mean to say it's on me to accept :)

Is this still something you guys think should make it into core? It's been a little while...

Automatically closed due to 14 days of inactivity.

I think the performance his is too much as per joern's comment:

http://jsfiddle.net/jboesch26/mmhwZ/4/

I've made some changes to make sure this doesn't affect performance that much:

https://github.com/jquery/jquery/pull/238

#8328 is a duplicate of this ticket.

I've spent some time with the newest patch and perf comparing it to the existing, I'm indifferent about the perf. My issue is this...

// Provided test...

var keys = jQuery.map( {a:1,b:2}, function( v, k ){
	return k;
});

equals( keys.join(""), "ab", "Map the keys from a hash to an array" );


// But this is totally bonkers:
var keys = jQuery.map( {a:1,b:2, length: 2 }, function( v, k ){
	return k;
});

equals( keys.join(""), "ab", "Map the keys from a hash to an array, has a length property" );

// > "01" 
// expected: "ab"

Reopening this for the current review

Updated pull request can be found here: https://github.com/jquery/jquery/pull/252

Did you look at the weird test results I posted previously?

Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!

Landed.

Why map to the object creates an array? I do not quite understand the history of this ticket, but the original patch was different behavior (map object to object) and I think it's more logical.

For an end use standpoint, you're right - but internally, worrying about something else being returned from map besides an array is out of scope for the patch. Even then, the patch isn't designed to be full featured by any means - just a simple way to parse out keys or values into an array.

That's not to say someone can't make a plugin to offer the object to object map - not sure how that'd look recursively though..