search

Incendo / cloud

Command framework & dispatcher for the JVM
https://cloud.incendo.org
MIT License
433 stars 54 forks source link

ArgumentPair should support TypeToken #711

Closed Machine-Maker closed 6 months ago

Machine-Maker commented 7 months ago

Currently, argumentpair only supports Pair<Class, Class> which causes some issues where I have a type with a generic (wildcard) as one of the types. There are probably issues with methods having the same signature if you just added overloads that took pairs of typetokens.

jpenilla commented 7 months ago

This could be solved with a TypePair/TypeTriple etc that can be created from classes or type tokens. But also I feel like removing ArgumentPair/ArgumentTriple in favor of specialized AggregateParser builders would be better...