gh-114894: add array.array.clear() method by mikeziminio · Pull Request #114919 · python/cpython · GitHub

Issue: Add array.clear() #114894

This will also need new unit tests (probably in test_array.py but I haven't checked) and a documentation entry (in array.rst)

This will also need new unit tests (probably in test_array.py but I haven't checked) and a documentation entry (in array.rst)

Thanks for the comment. I'll try to do it now.

done

@JelleZijlstra Jelle, I've added tests and documentation)

Can you also add this to the What's New document for Python 3.13 (Doc/whatsnew/3.13.rst)?

This is a bit of an indirect test, why not instead test that len(a) == 0?

All the testing logic from the test_array.py file works so that the methods are tested for all possible typecode.

To be honest, initially I used array comparison expecting that there would be a typecode check, but then I looked at array_richcompare() and realized that comparing arrays using the == operator is not suitable for checking the same typecode.

Therefore, I added two checks - the len() you suggested and the typecode comparison (just in case of future optimizations).

Add a test that an array with exported buffers cannot be cleared, something like this (untested):

with memoryview(a):
    with self.assertRaises(BufferError):
        a.clear()

done

In my personal view, according to the original issue, the intention behind adding the clear method is to allow array.array to implement collections.abc.MutableSequence, so we can add a test to test it and ensure we don't break it in the future.

There is already a relevant test case in

cpython/Lib/test/test_collections.py

Line 1968 in 28bb296

def test_MutableSequence(self):

, so what we need to do is simply include array.array in this existing test case.

Co-authored-by: AN Long <aisk@users.noreply.github.com>

…F-dSd.rst Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>

In my personal view, according to the original issue, the intention behind adding the clear method is to allow array.array to implement collections.abc.MutableSequence, so we can add a test to test it and ensure we don't break it in the future.

There is already a relevant test case in

cpython/Lib/test/test_collections.py

Line 1968 in 28bb296

def test_MutableSequence(self):

, so what we need to do is simply include array.array in this existing test case.

done

I added only the issubclass check (without isinstance), because for isinstance it would have to be check all typecodes.

Nitpick: These imports are basically sorted alphabetically, so this line may move to the first line.

done

Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>

@JelleZijlstra @aisk Guys, if I understand correctly, I made all the changes according to the reviews.
Please tell me what further steps are needed to merge the PR.

LGTM, I think the next step is to wait for a core member to approve and merge it.

Thanks all for the efforts, sorry for tagging but gentle nudge to @vstinner who seems to have touched the relevant files recently (according to BLAME).

Thanks!

pythongh-114894 : add array.array.clear() method

be9733a

bedevere-app Bot added the awaiting review label Feb 2, 2024

bedevere-app Bot mentioned this pull request Feb 2, 2024

Add array.clear() #114894

Closed

📜🤖 Added by blurb_it.

9c2c360

JelleZijlstra reviewed Feb 2, 2024

View reviewed changes

mikeziminio force-pushed the fix-issue-114894 branch from 3368a6b to 9c2c360 Compare February 2, 2024 18:51

mikeziminio added 2 commits February 2, 2024 22:54

added test and documentation

8adac42

fixed - trim trailing whitespace

e04c2fa

JelleZijlstra self-requested a review February 2, 2024 19:28

JelleZijlstra reviewed Feb 3, 2024

View reviewed changes

aisk reviewed Feb 3, 2024

View reviewed changes

Comment thread Modules/arraymodule.c Outdated

mikeziminio and others added 4 commits February 6, 2024 09:32

Update Modules/arraymodule.c

7630069

Co-authored-by: AN Long <aisk@users.noreply.github.com>

Update Misc/NEWS.d/next/Library/2024-02-02-15-50-13.gh-issue-114894.D…

556bf1a

…F-dSd.rst Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>

add memoryview test, update other tests

b5b9e63

add MutableSequence test

83422b0

mikeziminio requested a review from rhettinger as a code owner February 6, 2024 12:11

mikeziminio added 2 commits February 6, 2024 16:14

resolve clinic conflicts

4d5fcfb

add whatsnew doc

519ad21

mikeziminio requested review from JelleZijlstra and aisk February 6, 2024 13:28

change issue reference

6242242

aisk reviewed Feb 6, 2024

View reviewed changes

move so that it matches the alphabetical order.

fc76d73

rhettinger removed their request for review February 6, 2024 16:16

JelleZijlstra reviewed Feb 7, 2024

View reviewed changes

Comment thread Doc/whatsnew/3.13.rst Outdated

Comment thread Doc/library/array.rst

mikeziminio and others added 2 commits February 7, 2024 07:19

Update Doc/whatsnew/3.13.rst

7b46716

Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>

Update Doc/library/array.rst

a399ac6

Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>

JelleZijlstra approved these changes Feb 10, 2024

View reviewed changes

bedevere-app Bot added awaiting merge and removed awaiting review labels Feb 10, 2024

JelleZijlstra merged commit 9d1a353 into python:main Feb 10, 2024

bedevere-app Bot removed the awaiting merge label Feb 10, 2024

+                      a = array.array(self.typecode, self.example)
+                      self.assertRaises(TypeError, a.clear, 42)
+                      a.clear()
+                      self.assertEqual(a, array.array(self.typecode))

               from random import choice, randrange
               from itertools import product, chain, combinations
               import string
+              import array

                   return (PyObject *)np;
               }
+              /*[clinic input]

+                      a.append(self.example[2])
+                      a.append(self.example[3])
+                      self.assertEqual(a, array.array(self.typecode, self.example[2:4]))

gh-114894: add array.array.clear() method by mikeziminio · Pull Request #114919 · python/cpython · GitHub

Uh oh!

Uh oh!

gh-114894: add array.array.clear() method - #114919

gh-114894: add array.array.clear() method#114919JelleZijlstra merged 14 commits intopython:mainpython/cpython:mainfrom mikeziminio:fix-issue-114894mikeziminio/cpython:fix-issue-114894Copy head branch name to clipboard

Conversation

mikeziminio commented Feb 2, 2024 • edited by bedevere-app Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

mikeziminio commented Feb 2, 2024

Uh oh!

JelleZijlstra left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

aisk commented Feb 3, 2024 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

mikeziminio commented Feb 6, 2024

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

mikeziminio commented Feb 8, 2024

Uh oh!

aisk commented Feb 8, 2024

Uh oh!

Jason-Y-Z commented Feb 10, 2024

Uh oh!

Jason-Y-Z commented Feb 10, 2024

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

gh-114894: add array.array.clear() method#114919
JelleZijlstra merged 14 commits into
python:mainfrom
mikeziminio:fix-issue-114894

mikeziminio commented Feb 2, 2024 •
edited by bedevere-app Bot
Loading

aisk commented Feb 3, 2024 •
edited
Loading