Swati Jaiswal added the comment:
Is there any requirement for further modification or it can be accepted?
--
___
Python tracker
<http://bugs.python.org/issue25
Swati Jaiswal added the comment:
Modified according to Serhiy's last comment.
--
Added file: http://bugs.python.org/file41621/iss_25913_3.patch
___
Python tracker
<http://bugs.python.org/is
Swati Jaiswal added the comment:
Thank you Senthil for the improvements, I'll try to make it better next time :)
--
___
Python tracker
<http://bugs.python.org/is
Changes by Swati Jaiswal :
Added file: http://bugs.python.org/file41456/iss_25822_4.patch
___
Python tracker
<http://bugs.python.org/issue25822>
___
___
Python-bugs-list m
Swati Jaiswal added the comment:
Please check this. Fixed according to previous comment.
--
keywords: +patch
nosy: +curioswati
Added file: http://bugs.python.org/file41428/iss_25930.patch
___
Python tracker
<http://bugs.python.org/issue25
Swati Jaiswal added the comment:
Can someone help with tests? I just need a pointer on how to do it.
--
keywords: +patch
Added file: http://bugs.python.org/file41394/iss_13317.patch
___
Python tracker
<http://bugs.python.org/issue13
Changes by Swati Jaiswal :
Added file: http://bugs.python.org/file41393/iss_25913_2.patch
___
Python tracker
<http://bugs.python.org/issue25913>
___
___
Python-bugs-list m
Changes by Swati Jaiswal :
--
keywords: +patch
Added file: http://bugs.python.org/file41377/iss_25913.patch
___
Python tracker
<http://bugs.python.org/issue25
Swati Jaiswal added the comment:
@Andrew Barnert,
sorry, I didn't get your previous messages so please ignore the last message i
sent. I got your point i.e. We just need to provide the TypeError in the
Mapping. And the work around is never implemented.
Should I go for the patch wi
Swati Jaiswal added the comment:
Okay, so should I go for a patch for it? And sorry if it sounds naive, but do
we provide the work around or the user would implement if they purposely want
it. If we provide it, then where should it be written
Swati Jaiswal added the comment:
But the work around suggested here as:
def __reversed__(self):
return (self[k] for k in reversed(range(len(self
is also not a general solution, i.e. it is applicable for the following case:
m = MyDict({2:40, 0:10, 1:20})
but for any other mapping
Swati Jaiswal added the comment:
> If you do `reversed(d)`, you get a nice `TypeError: argument to reversed()
> must be a sequence`. But if you do `reversed(m)`, you get a reversed`
> iterator. And when you iterate it, presumably expecting to get 0 and 1 in
> some arbitrary order,
Swati Jaiswal added the comment:
Can it be reproduced in default branch? I tried but got:
AttributeError: module 'collections' has no attribute 'abc'
--
nosy: +curioswati
___
Python tracker
<http://bug
Changes by Swati Jaiswal :
Added file: http://bugs.python.org/file41300/iss_25822_3.patch
___
Python tracker
<http://bugs.python.org/issue25822>
___
___
Python-bugs-list m
Changes by Swati Jaiswal :
Added file: http://bugs.python.org/file41290/iss_25822_2.patch
___
Python tracker
<http://bugs.python.org/issue25822>
___
___
Python-bugs-list m
Swati Jaiswal added the comment:
Here is the patch, please review it. Do I need to write any test?
--
keywords: +patch
Added file: http://bugs.python.org/file41283/iss_25822.patch
___
Python tracker
<http://bugs.python.org/issue25
Swati Jaiswal added the comment:
I can help with this. Should I propose the changes I am going to make before
making them or just create the patch?
--
nosy: +curioswati
___
Python tracker
<http://bugs.python.org/issue25
Swati Jaiswal added the comment:
I want to work on this issue. @lac, can you please help as I searched but
couldn't find the related files. Where can I find the code for this?
--
nosy: +curioswati
___
Python tracker
<http://bugs.py
Swati Jaiswal added the comment:
are you working on it @eric?
--
nosy: +curioswati
___
Python tracker
<http://bugs.python.org/issue13317>
___
___
Python-bug
19 matches
Mail list logo
