From c29a1d2632a2d6c5d8ebf59b635372adde2f95ec Mon Sep 17 00:00:00 2001
From: eldpswp99 <nrg1392@naver.com>
Date: Thu, 12 Aug 2021 22:11:46 +0900
Subject: [PATCH 1/3] remove improper setter of deque.maxlen

before: maxlen was writable
        d = deque()
        d.maxlen = 10 was allowed
after : make maxlen read-only
	the before code raise attributError
---
 Lib/test/test_deque.py       | 2 --
 vm/src/stdlib/collections.rs | 5 -----
 2 files changed, 7 deletions(-)

diff --git a/Lib/test/test_deque.py b/Lib/test/test_deque.py
index 27e78e103..8e687adca 100644
--- a/Lib/test/test_deque.py
+++ b/Lib/test/test_deque.py
@@ -109,8 +109,6 @@ class TestBasic(unittest.TestCase):
         d.extendleft(it)
         self.assertEqual(list(it), [])
 
-    # TODO: RUSTPYTHON
-    @unittest.expectedFailure
     def test_maxlen_attribute(self):
         self.assertEqual(deque().maxlen, None)
         self.assertEqual(deque('abc').maxlen, None)
diff --git a/vm/src/stdlib/collections.rs b/vm/src/stdlib/collections.rs
index 6717e408c..301bbe4be 100644
--- a/vm/src/stdlib/collections.rs
+++ b/vm/src/stdlib/collections.rs
@@ -252,11 +252,6 @@ mod _collections {
             self.maxlen.load()
         }
 
-        #[pyproperty(setter)]
-        fn set_maxlen(&self, maxlen: Option<usize>) {
-            self.maxlen.store(maxlen);
-        }
-
         #[pymethod(magic)]
         fn getitem(&self, idx: isize, vm: &VirtualMachine) -> PyResult {
             let deque = self.borrow_deque();

From 8e6a2660fc3c411ec5f2c75681ef32a8ceebad46 Mon Sep 17 00:00:00 2001
From: eldpswp99 <nrg1392@naver.com>
Date: Thu, 12 Aug 2021 22:22:39 +0900
Subject: [PATCH 2/3] change deque.maxlen type from atomicCell<Option> to
 Option

---
 vm/src/stdlib/collections.rs | 20 +++++++++-----------
 1 file changed, 9 insertions(+), 11 deletions(-)

diff --git a/vm/src/stdlib/collections.rs b/vm/src/stdlib/collections.rs
index 301bbe4be..b9cb755d3 100644
--- a/vm/src/stdlib/collections.rs
+++ b/vm/src/stdlib/collections.rs
@@ -20,7 +20,7 @@ mod _collections {
     #[derive(Debug)]
     struct PyDeque {
         deque: PyRwLock<VecDeque<PyObjectRef>>,
-        maxlen: AtomicCell<Option<usize>>,
+        maxlen: Option<usize>,
     }
 
     type PyDequeRef = PyRef<PyDeque>;
@@ -76,7 +76,7 @@ mod _collections {
         ) -> PyResult<PyRef<Self>> {
             let py_deque = PyDeque {
                 deque: PyRwLock::default(),
-                maxlen: AtomicCell::new(maxlen),
+                maxlen,
             }
             .into_ref_with_type(vm, cls)?;
             if let OptionalArg::Present(iter) = iter {
@@ -88,7 +88,7 @@ mod _collections {
         #[pymethod]
         fn append(&self, obj: PyObjectRef) {
             let mut deque = self.borrow_deque_mut();
-            if self.maxlen.load() == Some(deque.len()) {
+            if self.maxlen == Some(deque.len()) {
                 deque.pop_front();
             }
             deque.push_back(obj);
@@ -97,7 +97,7 @@ mod _collections {
         #[pymethod]
         fn appendleft(&self, obj: PyObjectRef) {
             let mut deque = self.borrow_deque_mut();
-            if self.maxlen.load() == Some(deque.len()) {
+            if self.maxlen == Some(deque.len()) {
                 deque.pop_back();
             }
             deque.push_front(obj);
@@ -112,7 +112,7 @@ mod _collections {
         fn copy(&self) -> Self {
             PyDeque {
                 deque: PyRwLock::new(self.borrow_deque().clone()),
-                maxlen: AtomicCell::new(self.maxlen.load()),
+                maxlen: self.maxlen.clone(),
             }
         }
 
@@ -130,7 +130,7 @@ mod _collections {
         #[pymethod]
         fn extend(zelf: PyRef<Self>, iter: PyObjectRef, vm: &VirtualMachine) -> PyResult<()> {
             // TODO: use length_hint here and for extendleft
-            let max_len = zelf.maxlen.load();
+            let max_len = zelf.maxlen;
             let mut elements: Vec<PyObjectRef> = vm.extract_elements(&iter)?;
             if let Some(max_len) = max_len {
                 if max_len > elements.len() {
@@ -181,7 +181,7 @@ mod _collections {
         fn insert(&self, idx: i32, obj: PyObjectRef, vm: &VirtualMachine) -> PyResult<()> {
             let mut deque = self.borrow_deque_mut();
 
-            if self.maxlen.load() == Some(deque.len()) {
+            if self.maxlen == Some(deque.len()) {
                 return Err(vm.new_index_error("deque already at its maximum size".to_owned()));
             }
 
@@ -249,7 +249,7 @@ mod _collections {
 
         #[pyproperty]
         fn maxlen(&self) -> Option<usize> {
-            self.maxlen.load()
+            self.maxlen
         }
 
         #[pymethod(magic)]
@@ -287,7 +287,6 @@ mod _collections {
                     .collect::<Result<Vec<_>, _>>()?;
                 let maxlen = zelf
                     .maxlen
-                    .load()
                     .map(|maxlen| format!(", maxlen={}", maxlen))
                     .unwrap_or_default();
                 format!("deque([{}]{})", elements.into_iter().format(", "), maxlen)
@@ -315,14 +314,13 @@ mod _collections {
             let mul = sequence::seq_mul(&deque, n);
             let skipped = self
                 .maxlen
-                .load()
                 .and_then(|maxlen| mul.len().checked_sub(maxlen))
                 .unwrap_or(0);
 
             let deque = mul.skip(skipped).cloned().collect();
             PyDeque {
                 deque: PyRwLock::new(deque),
-                maxlen: AtomicCell::new(self.maxlen.load()),
+                maxlen: self.maxlen.clone(),
             }
         }
 

From 0e36ff83ec5785b1de1daebe1ffcc5775207ecfa Mon Sep 17 00:00:00 2001
From: eldpswp99 <nrg1392@naver.com>
Date: Thu, 12 Aug 2021 23:08:00 +0900
Subject: [PATCH 3/3] remove unnecessary Option<usize> clone

---
 vm/src/stdlib/collections.rs | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/vm/src/stdlib/collections.rs b/vm/src/stdlib/collections.rs
index b9cb755d3..dc87f6cde 100644
--- a/vm/src/stdlib/collections.rs
+++ b/vm/src/stdlib/collections.rs
@@ -112,7 +112,7 @@ mod _collections {
         fn copy(&self) -> Self {
             PyDeque {
                 deque: PyRwLock::new(self.borrow_deque().clone()),
-                maxlen: self.maxlen.clone(),
+                maxlen: self.maxlen,
             }
         }
 
@@ -320,7 +320,7 @@ mod _collections {
             let deque = mul.skip(skipped).cloned().collect();
             PyDeque {
                 deque: PyRwLock::new(deque),
-                maxlen: self.maxlen.clone(),
+                maxlen: self.maxlen,
             }
         }