Skip to content

Commit 1fc73b3

Browse files
authored
Clean up unsync in Base/CommonPlan (#308)
* Remove unsync and unsafe cells from BasePlan and CommonPlan * Remove unsafe impl Sync for CommonPlan and BasePlan * Remove the base_spaces feature * Fix a broken link in rustdoc
1 parent e750dfe commit 1fc73b3

File tree

9 files changed

+146
-181
lines changed

9 files changed

+146
-181
lines changed

Cargo.toml

+3-4
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,9 @@ default = []
4848
# -- Non mutally exclusive features --
4949

5050
# spaces
51-
base_spaces = []
52-
vm_space = ["base_spaces"]
53-
ro_space = ["base_spaces"]
54-
code_space = ["base_spaces"]
51+
vm_space = []
52+
ro_space = []
53+
code_space = []
5554

5655
# Run sanity GC
5756
sanity = []

docs/tutorial/code/mygc_semispace/global.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -115,7 +115,7 @@ impl<VM: VMBinding> Plan for MyGC<VM> {
115115

116116
// Modify
117117
// ANCHOR: prepare
118-
fn prepare(&self, tls: VMWorkerThread) {
118+
fn prepare(&mut self, tls: VMWorkerThread) {
119119
self.common.prepare(tls, true);
120120

121121
self.hi
@@ -129,7 +129,7 @@ impl<VM: VMBinding> Plan for MyGC<VM> {
129129

130130
// Modify
131131
// ANCHOR: release
132-
fn release(&self, tls: VMWorkerThread) {
132+
fn release(&mut self, tls: VMWorkerThread) {
133133
self.common.release(tls, true);
134134
self.fromspace().release();
135135
}

src/plan/gencopy/global.rs

+2-2
Original file line numberDiff line numberDiff line change
@@ -128,7 +128,7 @@ impl<VM: VMBinding> Plan for GenCopy<VM> {
128128
&*ALLOCATOR_MAPPING
129129
}
130130

131-
fn prepare(&self, tls: VMWorkerThread) {
131+
fn prepare(&mut self, tls: VMWorkerThread) {
132132
self.common.prepare(tls, true);
133133
self.nursery.prepare(true);
134134
if !self.in_nursery() {
@@ -140,7 +140,7 @@ impl<VM: VMBinding> Plan for GenCopy<VM> {
140140
self.copyspace1.prepare(!hi);
141141
}
142142

143-
fn release(&self, tls: VMWorkerThread) {
143+
fn release(&mut self, tls: VMWorkerThread) {
144144
self.common.release(tls, true);
145145
self.nursery.release();
146146
if !self.in_nursery() {

0 commit comments

Comments
 (0)