From 35a1b91c4b4153027a722a267ba0bcdfec6d2bfe Mon Sep 17 00:00:00 2001
From: John Kåre Alsaker <john.kare.alsaker@gmail.com>
Date: Thu, 31 Jan 2019 17:00:06 +0100
Subject: Address comments

---
 src/librustc_data_structures/jobserver.rs | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

(limited to 'src')

diff --git a/src/librustc_data_structures/jobserver.rs b/src/librustc_data_structures/jobserver.rs
index c85cdfbdcd8..48ac8125a0d 100644
--- a/src/librustc_data_structures/jobserver.rs
+++ b/src/librustc_data_structures/jobserver.rs
@@ -4,7 +4,7 @@ use std::sync::{Condvar, Arc, Mutex};
 use std::mem;
 
 #[derive(Default)]
-pub struct LockedProxyData {
+struct LockedProxyData {
     /// The number of free thread tokens, this may include the implicit token given to the process
     free: usize,
 
@@ -72,12 +72,15 @@ impl LockedProxyData {
 }
 
 #[derive(Default)]
-pub struct ProxyData {
+struct ProxyData {
     lock: Mutex<LockedProxyData>,
     cond_var: Condvar,
 }
 
-pub struct Proxy {
+/// A helper type which makes managing jobserver tokens easier.
+/// It also allows you to treat the implicit token given to the process
+/// in the same manner as requested tokens.
+struct Proxy {
     thread: Mutex<HelperThread>,
     data: Arc<ProxyData>,
 }
@@ -131,11 +134,11 @@ pub fn release_thread() {
 }
 
 impl Proxy {
-    pub fn release_token(&self) {
+    fn release_token(&self) {
         self.data.lock.lock().unwrap().release_token(&self.data.cond_var);
     }
 
-    pub fn acquire_token(&self) {
+    fn acquire_token(&self) {
         let mut data = self.data.lock.lock().unwrap();
         data.waiters += 1;
         if data.take_token(&self.thread) {
-- 
cgit 1.4.1-3-g733a5