slices.Delete

[Deleplace]

Hello, while researching for this Go proposal I noticed this code:

slices.Delete(cacheEntry, i, i)

which probably doesn't behave as intended and should look like this instead:

cacheEntry = slices.Delete(cacheEntry, i, i+1)

(2 differences: using the returned value, and deleting +1 element instead of zero).

I have not tried to execute it.

[andrewpollock]

Thank you so much for bringing this to my attention!

[Deleplace]

See my comment in #1646 : i+1 as last arg would be more appropriate