Use Vec::new instead of vec!

[randomPoison]

Using the vec! macro to create an empty Vec is a bit weird imo, generally I only see the macro used when you actually want to initialize the Vec with some values (like is done in the examples for this exercise). Students are more likely to use Vec::new, and I think that's the more idiomatic approach, so I think using Vec::new here would be better.

[djmitche]

I tend to use vec![] just because it's fewer characters, but you're right that doing so is unusual.

Also, bummer, I appear to not be able to merge things anymore.

[michael-kerscher]

There does not seem to be any official guideline for choosing between these two alternatives as far as I can find (rust-lang/style-team#70)

I also tend to use vec![] because this is shorter and it feels more similar to how I initialize a Vec with values.

let empty_function = Vec::new();
let empty_macro = vec![];
let values = vec![1,2,3];

But in the end I don't really mind how this is written and the code in the background is identical (if I disregard some fixed bugreports on different code being emitted for vec![] and Vec::new()...)

There is even a request for linting on vec![] usage but this was not done: rust-lang/rust-clippy#12774.