Require `Clone` instead of `Copy` for the key #14

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Open

msrd0 wants to merge 1 commit into mpdn:master from msrd0:clone-instead-of-copy

msrd0 commented Feb 21, 2025

No description provided.


          Require Clone instead of Copy for the key

ef2ee4f

msrd0 commented

View reviewed changes

src/lib.rs

    
                  #[inline]

                  pub fn top(&self) -> Option<K> {

                      self.top

                      self.top.clone()

Author

msrd0 Feb 21, 2025

I could instead change this to return a reference to the top if you want to break API

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet