Inconsistent behavior of the miniboxing plugin code transformations about miniboxing-plugin HOT 8 OPEN

@lastland yes, it's not yet automated. To be honest, seeing all the bugs MiniboxedFunctions triggered, I think it's best if we leave it that way. Do you think it's too difficult to make the replacements by hand?

Or, maybe, would putting the classes in the scala package directly make it easier?

from miniboxing-plugin.

I don't think that it would be more difficult than adding @miniboxeds by hand, but this could not be automated by using the minibox:mark-all compiler option, right?

from miniboxing-plugin.

No, it can't be automated. Now I'm see what you mean, the different transformations are inconsistent:

• @miniboxed annotations need to be introduced by hand, but can be automated via -P:minibox:mark-all
• Numeric and the other type classes have to be introduced by hand, but can't be automated via -P:minibox:mark-all
• Tuple accessors and Functions are automatically converted, and can only be turned off using -P:minibox:off.

I agree this causes confusion. What do you think would be a good approach?

from miniboxing-plugin.

I imagine it could all start from the @miniboxed annotation:

• Numeric and other type classes are replaced automatically by MiniboxedNumeric and co.
• Functions are replaced when their type arguments are marked as @miniboxed
• Tuple accessors are replaced when the two components are marked as @miniboxed

Then, if you don't add any @miniboxed annotation, nothing is changed. If you do, either manually or via -P:minibox:mark-all everything is consistently changed.

What do you think?

from miniboxing-plugin.

I agree. It feels most natural from a user's perspective (well, from my perspective 😜).

from miniboxing-plugin.

Thanks @lastland!
How would you feel about moving this to the next milestone? I don't think that I'm able to pull it off quickly and I really want to get to the 0.4 release out the door.

from miniboxing-plugin.

Of course. No worries. 😄

from miniboxing-plugin.

Cool, thanks @lastland! I will update the behavior for the next milestone. In the meantime, if you notice any strange behavior/need help getting the miniboxing plugin to do what you want, ask on gitter anytime.

from miniboxing-plugin.