burner/logger

static this constructor

Closed this issue · 6 comments

https://github.com/burner/logger/blob/master/std/logger.d#L771

Currently DMD forces us to insert a missing super call, which doesn't make sense for static this. It seems DMD think of it as a regular constructor.
I think this static this() module constructor should go out of this class. Being in the same module it will access private members likewise.

Yeah this is really not nice. I'm not sure if moving it out will make it any prettier or more corrent.

Regardless, I think DMD complaint about super() is a compiler bug.

I agree. Do you know if there is bugzilla entry for this.

Dunno. When in doubt, better file it.

the fix is easy. put the @trusted first