kevinresol / hxgenjs

Extensible JS generator for Haxe
57 stars 16 forks source link

Dce & abstracts #18

Closed benmerckx closed 6 years ago

benmerckx commented 6 years ago

Seems like hxgenjs generates output for all abstracts even if their implementation is unused (even with -dce full).

image

kevinresol commented 6 years ago

Are they empty? or at least no meaningful contents (fields / methods etc)

Ignore that.

The type list comes from the compiler. But I guess we can maintain a global "used type" list ourselves.

benmerckx commented 6 years ago

If there's no properties or statics at all they don't need to be generated? Since abstracts are otherwise a compile time feature the type itself shouldn't be needed (I think). Does the generator know it's an abstract?

benmerckx commented 6 years ago

There's still some left in my output without any statics, so I'm going to reopen this and investigate. They must be reaching the type processor in another way.

image