[issue10516] Add list.clear() and list.copy()
Roundup Robot devn...@psf.upfronthosting.co.za added the comment: New changeset 958a98bf924e by Eli Bendersky in branch 'default': updated whatsnew/3.3.rst with the new methods added to list and bytearray (issue 10516) http://hg.python.org/cpython/rev/958a98bf924e -- nosy: +python-dev ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: Committed the bytearray methods in revision 88733. Closing this issue. Will handle wrong exception and MutableSequence ABC method addition in separate issues. -- status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Raymond Hettinger rhettin...@users.sourceforge.net added the comment: Hmm, shouldn't self.__class__(self) be a good default implementation of copy()? I'd expect any sequence to support this way of creation from another sequence, even if it's inefficient. The copy() method isn't being proposed for MutableSequence because it presumes that we know something about the constructor's signature. For example, the constructor of array() needs the element storage type as an argument. We refuse the temptation to guess. In the Set api, we had no choice because many set-methods necessarily create a new set. To handle the constructor signature problem, the creation step was factored-out into the from_iterable() method so that a user could override it if necessary. Also copy() is handled oddly in the builtin types. To handle the constructor signature issue for subclasses, they ignore the subclass and return a instance of the base class. For example, the inherited copy() method on a subclass of list or dict will create an instance of list or dict, not of the subclass itself. Accordingly, subclasses that want instances of themselves have to override the inherited copy() method. They would have to do this anyway if they subclass contained any other data in the class dictionary that would need to be passed along to a copy. In short, we're better-off not supplying copy() as part of the MutableSequence ABC. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Raymond Hettinger rhettin...@users.sourceforge.net added the comment: The patch is fine. Do consider using assertIsNot() in the tests. Then go ahead and apply it. The OverflowError in bytearray.pop() is a bug, please open a separate report for it and make a patch changing it to IndexError. Assign that bug report to me. Go ahead and propose a patch for MutableSequence.clear() implemented with MutableSequence.pop() and catching an IndexError when empty. Thanks for your efforts. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Georg Brandl ge...@python.org added the comment: Hmm, shouldn't self.__class__(self) be a good default implementation of copy()? I'd expect any sequence to support this way of creation from another sequence, even if it's inefficient. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: Attaching a patch adding copy() and clear() to bytearrays, with tests and doc. I didn't add the methods to MutableSequence because I have a doubt about it - in particular which exception get raised by .pop if it's empty. Curiously, lists and bytearrays raise different exceptions in this case - IndexError and OverflowError, respectively. -- assignee: eli.bendersky - rhettinger stage: commit review - patch review status: closed - open Added file: http://bugs.python.org/file20905/issue10516.bytearray.1.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Georg Brandl ge...@python.org added the comment: Reading clear and copy are not supported by bytearray: shouldn't they be? (sort probably really makes no sense on bytearrays.) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: On Fri, Feb 25, 2011 at 10:11, Georg Brandl rep...@bugs.python.org wrote: Georg Brandl ge...@python.org added the comment: Reading clear and copy are not supported by bytearray: shouldn't they be? Perhaps they should, and it's not a big deal to implement. But I'm not 100% clear on the policy of how such changes are approved. Should this be discussed in the list? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Raymond Hettinger rhettin...@users.sourceforge.net added the comment: Unless someone raises a controversial and non-trivial issue about adding clear() and copy() to bytearray, there is no need for a python-dev discussion on the subject. Just post a patch to the tracker. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Georg Brandl ge...@python.org added the comment: Yes, it should be discussed on python-dev. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Georg Brandl ge...@python.org added the comment: In any case, this issue can be closed. -- status: open - closed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Boštjan Mejak bostjan.me...@gmail.com added the comment: I have installed Python 3.2 final on my Windows machine and I get an exception when doing list.copy or list.clear in the interpreter. Why is that so? -- Added file: http://bugs.python.org/file20886/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___I have installed Python 3.2 final on my Windows machine and I get an exception when doing list.copy or list.clear in the interpreter. Why is that so? ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Georg Brandl ge...@python.org added the comment: Because they got added *after* 3.2 was released? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Boštjan Mejak bostjan.me...@gmail.com added the comment: Right, right. My bad. Can't wait for Python 3.3! ;) -- Added file: http://bugs.python.org/file20887/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___Right, right. My bad. Can#39;t wait for Python 3.3! ;) ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Raymond Hettinger rhettin...@users.sourceforge.net added the comment: Georg, what is the issue? Is there some reason that bytearrays should not be copied or cleared? Is there some reason to prefer the current: dup = b[:] # copy del b[:] # clear -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Georg Brandl ge...@python.org added the comment: No -- but the question is if copy() and clear() mightn't be added to the (mutable) sequence ABC if we make all builtin such sequences implement them. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Raymond Hettinger rhettin...@users.sourceforge.net added the comment: The ABCs are subset of the methods for the concrete APIs. We've avoided the likes of copy() because it requires knowledge of the constructor's signature -- for example, MutableMapping does not cover copy(). It is okay for Eli to add MutableSequence.clear() because it can be implemented in terms of pop(), much like we do for MutableMapping.clear(). Eli, feel free to create a patch to add clear() and copy() to bytearray and to add clear() to MutableSequence. Assign the patch to me for review. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: A slightly revised patch committed in revision 88554: 1. Fixed Éric's whitespace comment 2. Fixed a test in test_descrtut.py which was listing list's methods 3. Moved the change to collections.py onto Lib/collections/__init__.py 4. Added NEWS entry Éric - as I mentioned earlier in this issue, I chose to leave the syntax of the docstring for the new methods similar to the same methods in dict (dict docstring look better and more internally consistent anyhow). I propose to move further discussion of this matter into a separate issue which will deal with the overall (in)consistency in the docstrings of list and dict. -- stage: patch review - commit review ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Raymond Hettinger rhettin...@users.sourceforge.net added the comment: The patch looks great. Please apply it. -- assignee: rhettinger - eli.bendersky resolution: later - accepted ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Ray.Allen ysj@gmail.com added the comment: Please modify the patch so that it can be applied to current py3k trunk cleanly. (Notice that Lib/collections.py has been changed to a package in #11085). -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Éric Araujo mer...@netwok.org added the comment: Ray: Eli can just refresh his patch and commit. Note that the patch program will prompt you for a file name if it can’t find the file for a diff hunk, so it should be trivial to apply a patch across a rename. Eli: Would you mind changing two nits before committing the nice patch? +L.clear() - None -- remove all items from L); It looks like other methods that return None just omit the “- type” part. + + That’s one new line too many. Some trailing spaces are also included. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Raymond Hettinger rhettin...@users.sourceforge.net added the comment: Eli doesn't need to post a new patch. I'm sure he will fix any nits in his commit. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: On Thu, Feb 24, 2011 at 05:26, Éric Araujo rep...@bugs.python.org wrote: Éric Araujo mer...@netwok.org added the comment: Ray: Eli can just refresh his patch and commit. Note that the patch program will prompt you for a file name if it can’t find the file for a diff hunk, so it should be trivial to apply a patch across a rename. Eli: Would you mind changing two nits before committing the nice patch? +L.clear() - None -- remove all items from L); It looks like other methods that return None just omit the “- type” part. + + That’s one new line too many. Some trailing spaces are also included. I will fix and commit over the weekend. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Eli Bendersky eli...@gmail.com: -- nosy: +ncoghlan ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Raymond Hettinger rhettin...@users.sourceforge.net added the comment: [Éric Araujo] +L.clear() - None -- remove all items from L); It looks like other methods that return None just omit the “- type” part. These kind of nitty comments really aren't helpful. It consumes more time to talk about them than they're worth. In this case, Eli was modeling after the docstring in dictobject.c: PyDoc_STRVAR(clear__doc__, D.clear() - None. Remove all items from D.); Just because list.remove.__doc__ failed to consistently follow that convention doesn't make Eli's patch incorrect. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Georg Brandl ge...@python.org: -- stage: needs patch - patch review ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Antoine Pitrou pit...@free.fr: -- stage: unit test needed - needs patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: Updated patch with versionadded tag for the new methods -- Added file: http://bugs.python.org/file20056/issue10516.5.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Ray.Allen ysj@gmail.com added the comment: That's good if it's so... can you explain why list_clear doesn't guarantee that the list is empty? Why would XDECREF populate the list? I don't quite understand it. Does this mean that durning the Py_DECREF progress the list may be populated again? It's not a problem. Here is the simplest example(with applying eli's patch): class A(object): def __init__(self, li): self._li = li def __del__(self): self._li.append(self) li = [] li.append(A(li)) li.clear() print(li) -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Ray.Allen ysj@gmail.com added the comment: eli, you should also add New in version 3.3 to the doc of the tow new list methods. -- nosy: +ysj.ray ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Boštjan Mejak bostjan.me...@gmail.com added the comment: mapp_methods ? Don't you mean map_methods ? -- Added file: http://bugs.python.org/file19954/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___span class=Apple-style-span style=font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; mapp_methods ?  Don#39;t you mean /spanspan class=Apple-style-span style=font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; map_methods ?/span ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Georg Brandl ge...@python.org added the comment: No, he means mapp_methods. Why don't you simply look at the file? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Boštjan Mejak bostjan.me...@gmail.com added the comment: mapp_methods looks like a typo. you know -- mapp_...? isn't map_... correct? -- Added file: http://bugs.python.org/file19957/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___mapp_methods looks like a typo. you know -- mapp_...? isn#39;t map_... correct? ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Brian Curtin cur...@acm.org added the comment: No, and please do not clutter this issue with any perceived typo discussions. -- nosy: +brian.curtin ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Éric Araujo mer...@netwok.org: Removed file: http://bugs.python.org/file19954/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Éric Araujo mer...@netwok.org: Removed file: http://bugs.python.org/file19957/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Boštjan Mejak bostjan.me...@gmail.com added the comment: Why mapp_methods, why not map_methods? Any reason for this? -- Added file: http://bugs.python.org/file19959/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___Why mapp_methods, why not map_methods? Any reason for this? ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Eric Smith e...@trueblade.com: Removed file: http://bugs.python.org/file19959/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Éric Araujo mer...@netwok.org added the comment: 1) Obviously because they’re mapping methods, not map methods. 2) Again, opening up the file and looking through it for some seconds or minutes would have allowed you to understand it. 3) Again, this is not the right place to discuss this. 4) Again, please do not send HTML email to this tracker. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Boštjan Mejak bostjan.me...@gmail.com added the comment: I'm troubled with one little letter: L.copy() - list -- a shallow copy of L); should be L.copy() - list -- shallow copy of L); without the letter 'a', because other sentences also don't say L.__sizeof__() -- *A* size of L in memory, in bytes); Please fix this. -- Added file: http://bugs.python.org/file19947/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___I#39;m troubled with one little letter:divspan class=Apple-style-span style=font-family: #39;Times New Roman#39;; font-size: medium; pre style=word-wrap: break-word; white-space: pre-wrap; quot;L.copy() -gt; list -- a shallow copy of Lquot;); should be/pre pre style=word-wrap: break-word; white-space: pre-wrap; span class=Apple-style-span style=font-family: #39;Times New Roman#39;; white-space: normal; pre style=word-wrap: break-word; white-space: pre-wrap; quot;L.copy() -gt; list -- shallow copy of Lquot;); without the letter #39;a#39;, because other sentences also don#39;t say quot;L.__sizeof__() -- *A* size of L in memory, in bytesquot;);/prepre style=word-wrap: break-word; white-space: pre-wrap; Please fix this./pre/span/pre/span/div ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Boštjan Mejak bostjan.me...@gmail.com added the comment: Can you please help me find the definition of the copy() method of dict in the Python sources? I want to see how that method is defined and compare the definition to the one in Eli's patch. -- Added file: http://bugs.python.org/file19948/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___Can you please help me find the definition of the copy() method of dict in the Python sources? I want to see how that method is defined and compare the definition to the one in Eli#39;s patch. ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Mark Dickinson dicki...@gmail.com: Removed file: http://bugs.python.org/file19947/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Mark Dickinson dicki...@gmail.com: Removed file: http://bugs.python.org/file19948/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Terry J. Reedy tjre...@udel.edu added the comment: Objects/dictobject.c -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment:
Boštjan,
a shallow copy: I took this directly from the documentation of dicts, which
says:
D.copy() - a shallow copy of D)
As I mentioned in an earlier message, the doc-strings of list and dict methods
are inconsistent in more than one way, so I'm going to leave this decision to
the committer. I'll be happy to help with fixes too.
Re your other question, in the Python source root, dictionaries are mostly
implemented in Objects/dictobject.c - there's an array called mapp_methods that
lists the functions used to implement relevant methods. For copy() it lists:
{copy,(PyCFunction)dict_copy, METH_NOARGS,
So you need dict_copy. Note that it's just a wrapper (of another wrapper, by
the way) bit it's a good place to start. Arm yourself with an editor or IDE
with some code-searching capabilities.
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10516
___
___
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: Was list.copy() also approved? After all, there are many ways to achieve the same even now: 1. L[:] 2. list(L) 3. import copy and then copy.copy Especially re the last one: list.copy() can be deep or shallow, which one should it be? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: Also, where is the *official* place to document list objects and their methods? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Georg Brandl ge...@python.org added the comment: Yes, list.copy was also approved IIRC. And it should be a shallow copy, like all other copy methods on builtins. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Boštjan Mejak bostjan.me...@gmail.com added the comment: This is really welcome. It makes Python even more readable. If 'a' is a list object, a[:] is not so obvious at first to a newcomer, but a.copy() is. Also, a.clear() is so perfect and understandable. I wish you could decorate Python versions prior to 3.3 with this two new list methods. -- nosy: +Retro ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: Attaching a patch with the following: 1. list.copy() and list.clear() implemented in Objects/listobject.c, with appropriate doc strings (modeled after dict's docstrings) 2. Same methods implemented in collections.UserList 3. Tests added that exercise the methods in both list and UserList Re the documentation, it's currently unclear where it should go. I asked on d...@python.org. -- Added file: http://bugs.python.org/file19938/issue10516.2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Éric Araujo mer...@netwok.org added the comment: Hi Eli, I think the right place is 4.6.4, http://docs.python.org/dev/library/stdtypes#mutable-sequence-types It starts with “List and bytearray objects support additional operations that allow in-place modification of the object”. For methods not supported by bytearray, you can use the fake footnote (8) and edit its texte (“sort() is not supported by bytearray objects”). Regards -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Raymond Hettinger rhettin...@users.sourceforge.net: -- assignee: - rhettinger keywords: +easy -patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: Following Éric's suggestion, I'm attaching an updated patch with with the documentation in Doc/library/stdtypes.rst updated with the new methods. There seems to be a slight Sphinx markup problem with this addition. I rewrote note (8) as: :meth:`clear`, :meth:`copy` and :meth:`sort` are not supported by :class:`bytearray` objects. Unfortunately, :meth:`copy` is getting linked to the copy module. -- keywords: +patch Added file: http://bugs.python.org/file19943/issue10516.3.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Eli Bendersky eli...@gmail.com: Removed file: http://bugs.python.org/file19835/issue10516.1.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Eli Bendersky eli...@gmail.com: Removed file: http://bugs.python.org/file19938/issue10516.2.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Raymond Hettinger rhettin...@users.sourceforge.net added the comment: Nothing will happen on this until 3.2 is done and the py3k branch starts with 3.3 submissions. -- resolution: - later ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Éric Araujo mer...@netwok.org added the comment: Eli, I learned this trick recently: :meth:`!copy`. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: Éric - thanks, it works [attaching updated patch]. However, don't you think the core problem is a Sphinx bug we should report? Raymond - this happens after final 3.2 release (on Feb 05 2011 if it's on schedule), right? -- Added file: http://bugs.python.org/file19945/issue10516.4.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Eli Bendersky eli...@gmail.com: Removed file: http://bugs.python.org/file19943/issue10516.3.patch ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Eli Bendersky eli...@gmail.com: -- nosy: +eli.bendersky ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: Objects/listobject.c has a static function named list_clear used internally. Is it possible to just expose this function as a clear() method? One problem is that it has this strange comment in the end: /* Never fails; the return value can be ignored. Note that there is no guarantee that the list is actually empty at this point, because XDECREF may have populated it again! */ However, looking at the code I'm not sure the list can be cleared any more than the function does, and it actually deallocates the ob_item field of the list. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Xuanji Li xua...@gmail.com added the comment: Hi, I'm also looking at listobject.c also... if we want list.clear() to behave exactly like del list[], we may be able to just call list_ass_slice on the list. Similarly for list.copy which should behave like a=l[:] -- nosy: +xuanji ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: Hi, I'm also looking at listobject.c also... if we want list.clear() to behave exactly like del list[], we may be able to just call list_ass_slice on the list. Similarly for list.copy which should behave like a=l[:] Note that when executed to do 'del lst[:]' (i.e. with argument v set to 0 and ilow/ihigh to the maximal range of the list), list_ass_slice will just call list_clear anyway, which is a cue that this indeed is the right way to do it, despite the strange comment I mentioned in my message above. -- Added file: http://bugs.python.org/file19834/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___div dir=ltrbrdiv class=gmail_quoteblockquote class=gmail_quote style=margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex; br Hi, I#39;m also looking at listobject.c also... if we want list.clear() to behave exactly like del list[], we may be able to just call list_ass_slice on the list. Similarly for list.copy which should behave like a=l[:]br /blockquote/divbrNote that when executed to do #39;del lst[:]#39; (i.e. with argument v set to 0 and ilow/ihigh to the maximal range of the list), list_ass_slice will just call list_clear anyway, which is a cue that this indeed is the right way to do it, despite the strange comment I mentioned in my message above.br br/div ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Georg Brandl ge...@python.org added the comment: Yes, list_clear should be called, but no, it cannot be used directly because a method needs a PyObject* return value. So a wrapper method is needed that looks like listappend() does for list.append(). list_copy() will just look like list_slice() with the index fiddling removed. -- nosy: +georg.brandl ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Xuanji Li xua...@gmail.com added the comment: That's good if it's so... can you explain why list_clear doesn't guarantee that the list is empty? Why would XDECREF populate the list? I don't quite understand it. eli: are you writing a patch for this? -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment: Georg, Thanks. Of course it should be wrapped like the others :-) Xuanji, Yes, I will try to get in something preliminary today. -- ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Eli Bendersky eli...@gmail.com added the comment:
Attaching a patch for list.clear():
1. Implements a new function in Objects/listobject.c named listclear() (to be
consistent with the other method functions)
2. listclear() is registered in list_methods and just calls list_clear(),
returning None
3. A documentation string is modeled after dict.clear(), but shaped a bit
differently to follow the conventions of other list method docs.
If this look fine to the more experienced devs, things left to do are:
1. Add tests
2. Implement the .copy() method in a similar manner + tests for it
Some random observations:
1. The naming of functions/methods could be made more consistent. See, for
example, list_reversed vs. listreverse.
2. The documentation style of list and dict methods is different for no
apparent reason:
help({}.pop) gives:
pop(...)
D.pop(k[,d]) - v, remove specified key and return the corresponding value.
If key is not found, d is returned if given, otherwise KeyError is raised
While help([].pop) gives:
pop(...)
L.pop([index]) - item -- remove and return item at index (default last).
Raises IndexError if list is empty or index is out of range.
Note the '--' which separates the signature from description in the list
version.
--
keywords: +patch
Added file: http://bugs.python.org/file19835/issue10516.1.patch
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue10516
___
___
Python-bugs-list mailing list
Unsubscribe:
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Eli Bendersky eli...@gmail.com: Removed file: http://bugs.python.org/file19834/unnamed ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Eli Bendersky eli...@gmail.com: -- Removed message: http://bugs.python.org/msg122522 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Giampaolo Rodola' g.rod...@gmail.com: -- nosy: +giampaolo.rodola ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Changes by Éric Araujo mer...@netwok.org: -- nosy: +eric.araujo ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear()
New submission from Terry J. Reedy tjre...@udel.edu: Add list.clear() method with obvious semantics. Pro: 1. parallel to set/dict/defaultdict/deque.clear(), usable in generic mutable collection function; 2. makes it easier to switch between list and other collection class; 3. current alternatives are not as obvious; 4. some people seem to expect it. Anti: 1. unneeded; del l[:] or l[:]=[] do same already. Guido: (python-ideas list, 'Set Syntax' thread, today) FWIW I'm fine with adding list.clear() to 3.3. -- components: Interpreter Core messages: 122251 nosy: terry.reedy priority: normal severity: normal stage: unit test needed status: open title: Add list.clear() type: feature request versions: Python 3.3 ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear()
Eric Smith e...@trueblade.com added the comment: Guido's email is archived at: http://mail.python.org/pipermail/python-ideas/2010-November/008732.html -- nosy: +eric.smith ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
[issue10516] Add list.clear() and list.copy()
Raymond Hettinger rhettin...@users.sourceforge.net added the comment: Guido approved these both in a thread earlier this year. The reasoning for copy() was the same as for clear(), some folks couldn't cope with: b = a[:] -- nosy: +rhettinger title: Add list.clear() - Add list.clear() and list.copy() ___ Python tracker rep...@bugs.python.org http://bugs.python.org/issue10516 ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com
