TR suggestions

master
Andrew Johnson 6 years ago
parent f954e1b30a
commit 536ea812a2

@ -42,13 +42,14 @@ pub fn toErrorCode(i: i32) -> ErrorCode {
} }
} }
pub struct AuthorizedSessionRaw(*const c_void); struct AuthorizedSessionInner(*const c_void);
#[derive(Clone)] #[derive(Clone)]
pub struct AuthorizedSession pub struct AuthorizedSession
{ {
session: Rc<AuthorizedSessionRaw> session: Rc<AuthorizedSessionInner>
} }
impl Drop for AuthorizedSessionRaw { impl Drop for AuthorizedSessionInner {
fn drop(&mut self) { fn drop(&mut self) {
unsafe { unsafe {
magic::free_override_session(self.0); magic::free_override_session(self.0);
@ -65,7 +66,7 @@ pub fn authorize_override() -> Result<AuthorizedSession,ErrorCode> {
magic::poll_override_session() magic::poll_override_session()
}; };
let session = AuthorizedSession { let session = AuthorizedSession {
session: Rc::new(AuthorizedSessionRaw(session)) session: Rc::new(AuthorizedSessionInner(session))
}; };
check_error(session) check_error(session)
} }
@ -79,7 +80,7 @@ pub fn authorize_privileged() -> Result<AuthorizedSession,ErrorCode> {
magic::poll_physical_override_privileged_session() magic::poll_physical_override_privileged_session()
}; };
let session = AuthorizedSession { let session = AuthorizedSession {
session: Rc::new(AuthorizedSessionRaw(session)) session: Rc::new(AuthorizedSessionInner(session))
}; };
check_error(session) check_error(session)
} }
@ -93,7 +94,7 @@ pub fn authorize_admin() -> Result<AuthorizedSession,ErrorCode> {
magic::poll_physical_override_admin_session() magic::poll_physical_override_admin_session()
}; };
let session = AuthorizedSession { let session = AuthorizedSession {
session: Rc::new(AuthorizedSessionRaw(session)) session: Rc::new(AuthorizedSessionInner(session))
}; };
check_error(session) check_error(session)
} }

Loading…
Cancel
Save