`SugarApplicationException`s aren't passed to `IMixinErrorHandler`

[melontini]

While testing IMixinErrorHandler on Connector, I noticed that Sugar exceptions aren't passed to the handler.

The Mixin in question: [link]

The IMixinErrorHandler in question: [link]

[LlamaLad7]

IMixinErrorHandler is very broken generally, e.g. no injection check failures get passed to it. Do you have a particular need for sugar exceptions to be passed to it exclusively? If the underlying behaviour in mixin were improved you would be able to get them by looking at the suppressed exceptions from the injection check failure.

[melontini]

Ah, wasn't aware of issues with the handler, thank you. (I guess this is what SpongePowered/Mixin#640 is about?)

Anyway, there's no particular need for sugar exceptions. I just assumed this was something to do with sugar exclusively. Sorry.

I'll close this, as this is not an issue with MixinExtras.

[LlamaLad7]

Yeah, that getting merged would be ideal.