Errors are suppressed in generator comprehensions

[ncw]

>>> list(i for x in range(10))
[]

Which should have output

>>> list(i for x in range(10))
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "<stdin>", line 1, in <genexpr>
NameError: name 'i' is not defined

[corona10]

@ncw I will take a look this issue

[corona10]

list(i for x in range(10))
-> Expression(body=Call(func=Name(id='list', ctx=Load()), args=[GeneratorExp(elt=Name(id='i', ctx=Load()), generators=[comprehension(target=Name(id='x', ctx=Store()), iter=Call(func=Name(id='range', ctx=Load()), args=[Num(n=10)], keywords=[], starargs=None, kwargs=None), ifs=[])])], keywords=[], starargs=None, kwargs=None))

[i for x in range(10)]
-> Expression(body=ListComp(elt=Name(id='i', ctx=Load()), generators=[comprehension(target=Name(id='x', ctx=Store()), iter=Call(func=Name(id='range', ctx=Load()), args=[Num(n=10)], keywords=[], starargs=None, kwargs=None), ifs=[])]))

[ncw]

@corona10 are you getting anywhere with this? Do you need help?

[corona10]

@ncw

gpython/py/generator.go

Line 100 in 2609237

res, err := VmRunFrame(it.Frame)

Well, I tracked the code and I found that something issue in this side.
But I don't know which is the main reason for this issue yet.

So if you already solve this issue, Then please upload a PR for this issue :)
I can learn something from your PR.

[ncw]

Well, I tracked the code and I found that something issue in this side.
But I don't know which is the main reason for this issue yet.

I haven't figured it out either!

But yes, it is probably something to do with making a new Vm Frame and the errors being suppressed there rather than propagated.

[corona10]

@ncw
I found the main reason for this bug.
I will upload the PR soon

[ncw]

I found the main reason for this bug.
I will upload the PR soon

Great. Don't forget a failing test case :-)