Debug NodeRef Warning (#2414)

[martinfrances107]

This is a response to #2414

In production #[repr(transparent)] is use to ensure a compact representation in memory.
In developement the memory is slightly expanded.

Things I am thinking about

A) I have introduced a maintence hazard... the documentation block for NodeRef is repeated twice
So any future PRs will have to make sure things are kept in sync.
I would love to avoid that...

B) I may have been over eager.. ( adding #[track_caller ] in too many places. )
impl<T: ElementDescriptor> Clone for NodeRef {
#[track_caller]
fn clone(&self) -> Self {
*self
}

[gbj]

Thanks for working on this! I proposed a few small changes in the comments above. Much appreciated as always.

[martinfrances107]

Sorry for the late reply

There is a small nuance here.

This directive is dropped for a debug build
something I was hoping to avoid .

  #[repr(transparent)]

I have to do it by adding a

 #[allow(missing_docs)]

[martinfrances107]

I recently, rebased and force pushed - as I think it will fix build errors

[benwis]

To be honest I think the CI is just unhappy

[martinfrances107]

Ok this next version looks much less like a dogs breakfast

#[cfg_attr(not(debug_assertions), repr(transparent)]

That was the key to this, that I was not seeing ... thank you - thank you

[gbj]

Thanks very much for the contribution, and for your patience through all the back and forth!