Opened 11 years ago
Closed 3 years ago
#2275 closed enhancement (wontfix)
Move SCode.IMPORT
| Reported by: | Martin Sjölund | Owned by: | somebody |
|---|---|---|---|
| Priority: | high | Milestone: | 2.0.0 |
| Component: | Frontend | Version: | trunk |
| Keywords: | Cc: | Per Östlund, Adrian Pop |
Description
SCode.IMPORT is currently an SCode.Element with visibility information which is not needed. We could store the imports in a separate list (pretty-print always with imports at the top of the class) for efficiency.
Change History (9)
comment:1 by , 9 years ago
| Milestone: | 1.9.3 → 1.9.4 |
|---|
comment:5 by , 8 years ago
| Milestone: | 1.11.0 → 1.12.0 |
|---|
Milestone moved to 1.12.0 due to 1.11.0 already being released.
comment:7 by , 7 years ago
Replying to casella:
@perost is this still relevant?
You'll have to ask sjoelund.se about that. It's not very relevant to the new instantiation, although I guess it could be slightly more efficient.
comment:8 by , 7 years ago
I am a little bit torn now. The bootstrapping does treat protected import differently from public import...
comment:9 by , 3 years ago
| Resolution: | → wontfix |
|---|---|
| Status: | new → closed |
Considering that we'd have to rewrite a lot of code to handle this and there aren't any clear need for it I don't see much point in keeping this ticket open.
Moved to new milestone 1.9.4