Issue2233

CPython 2.7:

>>> {}.pop("a")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
KeyError: 'a'
>>> {"a":1}.pop("b")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
KeyError: 'b'


Jython 2.7:
>>> {}.pop("a")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
KeyError: 'popitem(): dictionary is empty'
>>> {"a":1}.pop("b")
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
KeyError: 'popitem(): dictionary is empty'

The right exception type was being thrown. It's just the message given to the exception instance that's wrong. Easily fixed. Attaching a patch for review.

Santoso,

+1. Do you want to apply this trivial patch?

D'oh. I'm still in sending patches for commit mindset. Is there any
developers guide for committers somewhere?

On Mon, Dec 15, 2014, 10:33 AM Jim Baker <report@bugs.jython.org> wrote:

>
> Jim Baker added the comment:
>
> Santoso,
>
> +1. Do you want to apply this trivial patch?
>
> ----------
> nosy: +zyasoft
>
> _______________________________________
> Jython tracker <report@bugs.jython.org>
> <http://bugs.jython.org/issue2233>
> _______________________________________
>

Nevermind, I found it!

The fix is now committed with https://hg.python.org/jython/rev/f68d65a141f7