Skip to content
This repository has been archived by the owner on Feb 14, 2021. It is now read-only.

Fix: Already borrowed: BorrowMutError. Don't borrow_mut() recursively #32

Merged
merged 3 commits into from
Aug 10, 2018

Conversation

lexfrl
Copy link
Contributor

@lexfrl lexfrl commented Aug 8, 2018

fixes #31
To do: add a test

Copy link
Collaborator

@NikVolf NikVolf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about make an example which utilizes fix?

src/external.rs Outdated
@@ -1,6 +1,7 @@
use std::any::Any;
use std::collections::HashMap;
use std::rc::Rc;
use std::cell::{RefCell};
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Redundant braces

@lexfrl lexfrl force-pushed the already-borrowed-error-fix branch from a7a44d0 to 9d331fe Compare August 9, 2018 16:55
@lexfrl
Copy link
Contributor Author

lexfrl commented Aug 9, 2018

Copy link
Collaborator

@NikVolf NikVolf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cool

@lexfrl lexfrl merged commit 7b729a0 into master Aug 10, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow inner calls
2 participants