Re: [pypy-dev] Ordered dict in PyPy

2015-01-11 Thread Armin Rigo
Hi all, Thanks for the feedback. It looks like the general opinion is to raise RuntimeError when detecting changes. I'll do that then. About '__reversed__', I suppose it should be implemented the same way, with an RPython-provided iterator which raises RuntimeError too. A bientôt, Armin. ___

Re: [pypy-dev] Ordered dict in PyPy

2015-01-11 Thread Alex Gaynor
I think we should avoid doing extras copies, it creates weird scenarios where the performance is randomly worse on PyPy, which can be very hard to debug. Alex On Sun Jan 11 2015 at 11:51:00 AM Maciej Fijalkowski wrote: > btw, I'm also ok with reversed being implemented by simply making a copy >

Re: [pypy-dev] Ordered dict in PyPy

2015-01-11 Thread Maciej Fijalkowski
btw, I'm also ok with reversed being implemented by simply making a copy On Sun, Jan 11, 2015 at 9:37 PM, Maciej Fijalkowski wrote: > I'm with Alex on that - raising RuntimeError is a good behavior when > "you're not supposed to do that" happens. I would go with 1) as > opposed to 2) > > On Sun,

Re: [pypy-dev] Ordered dict in PyPy

2015-01-11 Thread Samuel Villamonte
IMHO, there's a part of an old "saying": [...] Errors should never pass silently. Unless explicitly silenced. So I'd say better to have users deal with the RuntimeError, and document it in the CPython diferences page. 2015-01-11 14:26 GMT-05:00 Alex Gaynor : > IMO, it's clear that CPython inte

Re: [pypy-dev] Ordered dict in PyPy

2015-01-11 Thread Maciej Fijalkowski
I'm with Alex on that - raising RuntimeError is a good behavior when "you're not supposed to do that" happens. I would go with 1) as opposed to 2) On Sun, Jan 11, 2015 at 9:26 PM, Alex Gaynor wrote: > IMO, it's clear that CPython intends this to be "undefined behavior", > raising a RuntimeError i

Re: [pypy-dev] Ordered dict in PyPy

2015-01-11 Thread Alex Gaynor
IMO, it's clear that CPython intends this to be "undefined behavior", raising a RuntimeError is a perfectly acceptable undefined behavior IMO -- better than corrupting the data. For __eq__ and __reversed__ and popitem(last=False) we can just have functions in __pypy__ and call them from class Orde

Re: [pypy-dev] Ordered dict in PyPy

2015-01-11 Thread Armin Rigo
Hi Laura, On 11 January 2015 at 19:57, Laura Creighton wrote: > Can we talk the CPython developers into raising RunTimeError for > concurrent modifications? No, we can't expect them to change that: http://bugs.python.org/issue19414 shows they have no plan to have well-defined behavior (either Ru

Re: [pypy-dev] Ordered dict in PyPy

2015-01-11 Thread Laura Creighton
Can we talk the CPython developers into raising RunTimeError for concurrent modifications? Laura ___ pypy-dev mailing list pypy-dev@python.org https://mail.python.org/mailman/listinfo/pypy-dev