Fix recently-introduced Fdelete bug

Problem reported by Pip Cet in:
* src/fns.c (Fdelete): Fix correctness bug via a simpler (though more
memory-intensive) approach.  It’s probably not worth optimizing
the memory usage yere.
* test/src/fns-tests.el (test-vector-delete): Add test for the bug.
1 job for master in 1 minute and 13 seconds (queued for 3 seconds)
Status Job ID Name Coverage
failed #7071


Name Stage Failure
test-all Test
error: cannot lock ref 'refs/remotes/origin/master': Unable to create '/builds/emacs/emacs/.git/refs/remotes/origin/master.lock': File exists.

Another git process seems to be running in this repository, e.g.
an editor opened by 'git commit'. Please make sure all processes
are terminated then try again. If it still fails, a git process
may have crashed in this repository earlier:
remove the file manually to continue.
fatal: failed to run reflog
ERROR: Job failed: exit code 1