Auto merge of #6621 - Ms2ger:repeat, r=SimonSapin

Move away from the repeat().take().collect() pattern.

This was the preferred pattern between the deprecation of Vec::from_elem and
the addition of the count argument to the vec![] macro.

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/6621)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2015-07-14 02:42:26 -06:00
commit 556c0e1509
6 changed files with 7 additions and 16 deletions

View file

@ -5232,7 +5232,6 @@ class CGBindingRoot(CGThing):
'util::str::DOMString',
'std::borrow::ToOwned',
'std::cmp',
'std::iter::repeat',
'std::mem',
'std::num',
'std::ptr',
@ -5555,7 +5554,7 @@ class CallbackMember(CGNativeMember):
if self.argCount > 0:
replacements["argCount"] = self.argCountStr
replacements["argvDecl"] = string.Template(
"let mut argv = repeat(UndefinedValue()).take(${argCount}).collect::<Vec<_>>();\n"
"let mut argv = vec![UndefinedValue(); ${argCount}];\n"
).substitute(replacements)
else:
# Avoid weird 0-sized arrays