You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Follow-up from PR #57 review (CommandArg.cs:310): are there other built-in types worth
supporting natively in CommandArg.TryGet<T>, given the console's delimiter and quoting
model?
System.Numerics: Complex (re, im) — BigInteger is already supported, so this is
consistent.
Possibly Matrix4x4 and Vector4-based color formats — heavier, decide separately.
Caveats to settle before implementing
Text format per type must be documented in the command help surface and survive the
shell's delimiter set (,, ;, :, _, /, \) — e.g. Bounds uses : inside
Unity's own ToString(), which collides with the delimiter set; choose an
unambiguous format or accept Unity-ToString-style input like the Rect parser does
(x:/width: stripping).
Cheap types first; anything needing allocation-heavy parsing (AnimationCurve,
Gradient) is likely out of scope for a console argument.
Data-driven PlayMode coverage per type in CommandArgTests, matching the house
style, plus ParsingIsCultureInvariant cases where relevant.
Related: the TryGet dispatch refactor issue (table-driven registry) should land first or
together, so each new type is one table entry, not another else if arm.
DISCLOSURE: LLM-GENERATED TEXT
Follow-up from PR #57 review (CommandArg.cs:310): are there other built-in types worth
supporting natively in
CommandArg.TryGet<T>, given the console's delimiter and quotingmodel?
Current built-in coverage
float/double/decimal, BigInteger, string.
properties/const fields (
red,MaxValue).Candidates that parse naturally from delimited text
Bounds(center/size),BoundsInt(position/size),RectOffset(left/top/right/bottom),
Plane(normal + distance),Ray(origin + direction).Complex(re, im) — BigInteger is already supported, so this isconsistent.
Matrix4x4andVector4-based color formats — heavier, decide separately.Caveats to settle before implementing
shell's delimiter set (
,,;,:,_,/,\) — e.g. Bounds uses:insideUnity's own
ToString(), which collides with the delimiter set; choose anunambiguous format or accept Unity-ToString-style input like the Rect parser does
(
x:/width:stripping).CommandArgParsers, pinnedNumberStyles + InvariantCulture).
Gradient) is likely out of scope for a console argument.
CommandArgTests, matching the housestyle, plus
ParsingIsCultureInvariantcases where relevant.Related: the TryGet dispatch refactor issue (table-driven registry) should land first or
together, so each new type is one table entry, not another
else ifarm.