Ruby

https://github.com/ruby/ruby/pull/3768

What about a way to bulk add items, e.g. q.concat or whatever is the same for Array, and maybe it would be best to have the first argument as an array, e.g. Queue.new(worklist)? I think it's more expensive to expand it in CRuby when you write *worklist.

Internally, this might still be adding one item at a time, in order to invoke the right "wakeup" machinery.

I'm not worried about Queue.new(worklist) or Queue.new(*worklist), so that's fine if more people feel that way. I think the key thing is conciseness in text source code, and also avoiding needing to synchronise while adding each individual item.

That all makes sense to me.

I agree that the constructor should take an enumerable rather than variadic arguments, as it would be consistent with Set.new([1, 2, 3]), and Array.new([1, 2, 3])

Agreed with @byroot (Jean Boussier) (actually I was going to note the same, he beat me to it).

I would also like to note that different from Set, for example, the order of items in the supplied parameter matters in the Queue case. Even though the expected outcome is for the items to be pushed to the Queue in given order, it might still be a good idea to explicitly call that out in the method documentation.

I updated to take a single array rather than a variable number of arguments.

I had two choices for how to do this - Set takes an Enumerable, using each to access items, and Array takes another Array, using #to_ary if needed.

I went with the same as what Array does, because both Array and Queue are core libraries, where Set is a standard library.

I accept the idea. Take Enumerable as initial values.

Matz.