tree 968a79ab697bfa023cdc0a564597b4ba131d1a8d
parent 7ff86095f361ea5ef5f1810d85bf9d325f68a95e
author JOE1994 <joseph942010@gmail.com> 1590947174 -0400
committer JOE1994 <joseph942010@gmail.com> 1590947174 -0400

Replace 'clone_from_slice' with 'copy_from_slice'
Currently, 'shared_memory::safe_copy' uses 'clone_from_slice' to
perform memory copy. 'clone_from_slice' iterates over each element of
slice to invoke 'clone_from'. According to docs of 'clone_from_slice', it can be
more performant to use 'copy_from_slice' instead when target slice
implements 'Copy'(https://doc.rust-lang.org/std/primitive.slice.html#method.clone_from_slice).
Since '&[u8]' implements 'Copy', I thought replacing 'clone_from_slice'
with 'copy_from_slice' would be a helpful change regarding
performance.

Thank you for reviewing this PR :)