Feature: `Rc::clone_raw` (and for Arc)

[Diggsey]

When using from_raw/into_raw functions with Rc, you often want to obtain a new reference to a raw pointer, without taking ownership. At the moment you have to do this dance:

fn clone_raw<T>(ptr: *const T) -> Rc<T> {
    let result = unsafe { Rc::from_raw(ptr) };
    ::std::mem::forget(result.clone());
    result
}

This is quite error prone and makes little sense to anyone trying to read the code. It would be better if the standard library had clone_raw built in for Rc and Arc, and possibly for their weak variants.

[zakarumych]

This looks like a simple and useful feature, but seems forgotten.

[ast-ral]

I think a potentially easy solution to this is to just have a function to increment the strong count directly from the pointer, and then have the user make two Rcs from it later, using Rc::from_raw. The current code to increment the strong count on a raw Rc ptr is:

unsafe fn increment_count<T>(ptr: *const T) {
    let rc = Rc::from_raw(ptr);
    std::mem::forget(rc.clone());
    std::mem::forget(rc);
}

Which seems a bit silly to just increment one value.

[Mark-Simulacrum]

https://github.com/rust-lang/rust/pull/79285 stabilizes the increment/decrement operators.

[ast-ral]

That unfortunately only seems to apply to Arc. (Edit: Upon further review, Rc is mentioned as a follow-up change, sorry to bother.)