Yes, looks correct to me!

@craustin Can you give a concrete example of a %paste which fails? That way we can add it to a test routine.

Clipboard: """
print 'test'

"""
[not sure if this will come through in the comment, but the middle whitespace line contains two spaces]

Unfortunately, I cannot reproduce this with your example in the terminal in Ubuntu 12.04.

Can any other devs chime in as to whether this bit of code should even exist in the first place? i.e., is the correct fix instead just to remove this line?

I think it's aiming to cancel out the incremented prompt number when you hit return on an empty cell - but it's not actually triggered in that case, or we would have seen the error long ago. I suspect it can go, but I want to think through what's going on in different cases to convince myself it's not required.

@takluyver Any more thought as to whether or not the correct fix is as proposed or just removing the case? I cannot reproduce the exception and am not familiar with the logic of this section.

I'm pretty sure now that we can just remove it. See PR #2783.

@craustin Can you check out PR #2783 and run it though the test case that you had for this bug? Please also make sure that your prompt numbers don't change in unexpected ways.

Closed in favour of #2783.