"super" isn't recognized

poppopjmp / shedskin

Automatically exported from code.google.com/p/shedskin

0 stars 0 forks source link

"super" isn't recognized #93

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

What steps will reproduce the problem?
1. shedskin -e Shape.py
2. make
3. See that it built fine.
4. Edit Shape.py to remove the #{ #} comments.
5. shedskin -e Shape.py
6. make

What is the expected output? What do you see instead?

You'll get this error:

*ERROR* Shape.py:20: unbound identifier 'super'

You see that it works if I replace the builtin super with MySuper.

What version of the product are you using? On what operating system?

This morning's git, on Ubuntu 9.

Original issue reported on code.google.com by uran...@gmail.com on 19 Aug 2010 at 5:59

GoogleCodeExporter commented 8 years ago

thanks for the new issues! I will try to answer them within a few days, but 
unfortunately I will be away for the weekend, so hopefully before that..

Original comment by mark.duf...@gmail.com on 19 Aug 2010 at 8:04

GoogleCodeExporter commented 8 years ago

I'm not a big fan of 'super', but I guess we can usually simply rewrite it. I 
will have a look.

Original comment by mark.duf...@gmail.com on 19 Aug 2010 at 8:05

Changed state: Accepted

GoogleCodeExporter commented 8 years ago

(pushing this one to 0.7 or later)

Original comment by mark.duf...@gmail.com on 28 Sep 2010 at 10:01

GoogleCodeExporter commented 8 years ago

Original comment by mark.duf...@gmail.com on 2 Oct 2010 at 6:25

Added labels: Priority-Low
Removed labels: Priority-Medium

GoogleCodeExporter commented 8 years ago

I committed basic support for 'super' in GIT. it only works if you use 
'super(.., self).method(..)' syntax, but I guess this is the most common use 
case.. I don't think supporting 'super' further would be worth the effort, so 
I'd like to know if this change works for you, so we can close the issue.. ;)

Original comment by mark.duf...@gmail.com on 30 Oct 2010 at 5:26

GoogleCodeExporter commented 8 years ago

Original comment by mark.duf...@gmail.com on 12 Nov 2010 at 9:31

Changed state: Fixed

GoogleCodeExporter commented 8 years ago

Verified: it works for me.  Thanks.

Original comment by uran...@gmail.com on 17 Nov 2010 at 5:46

GoogleCodeExporter commented 8 years ago

thanks for checking!

Original comment by mark.duf...@gmail.com on 17 Nov 2010 at 11:13