Complex use of `Vec`/`VecDeque` `remove`

What it does

Detect uses of Vec::remove or VecDeque::remove when Vec::pop, VecDeque::pop_front or VecDeque::pop_back could have been used

Advantage

• Makes the code easier to read
• Possibly makes the code run faster

Drawbacks

• In the case of Vec::remove it would break code because it changes the return type from T to Option<T>

Example

vec.remove(vec.len() - 1);

vec_deque.remove(0);
vec_deque.remove(vec_deque.len() - 1);

Could be written as:

vec.pop();

vec_deque.pop_front();
vec_deque.pop_back();

Start by locating the existing Clippy lint entry points and tests for collection-method patterns, then compare how calls to Vec::remove and VecDeque::remove are analyzed. Use the examples in the issue to define the pop, pop_front, and pop_back cases, while checking the Vec::remove return-type drawback. Done means the intended cases are detected with tests covering the suggested replacements.