musictheory / NilScript

Objective-C-style language superset of JavaScript with a tiny, simple runtime
Other
50 stars 5 forks source link

3.0: Remove @forward #138

Closed iccir closed 6 years ago

iccir commented 6 years ago

Per the documentation from circa 1.x:

In general, the need to use @forward indicates an underlying issue with the dependency tree, which will cause issues if you need to use @const/@enum inlining or the squeezer.

@const and @enum have been inlined by default since 2.0. One of our assumptions with 3.0 is that the squeezer will always be used for release builds.

@forward no longer makes sense.

iccir commented 6 years ago

After this commit, Model.NSClass still has a forward property. We can probably consolidate this with placeholder.