✨New follow-up bitcoin-core change opprotunity✨
📢mentioned in: #32821 rpc: Handle -named argument parsing where '=' character is used
github.com/bitcoin/bitcoin/p…
"> I'm wondering if there's a different way we can go about this without adding another table of arguments that need special treatment.
I'm also not a fan of separate tables and suggested the following change to unify them earlier: b998cc52d51b48db9271fdba0bd69e9aaccb7999 ([tag](
github.com/ryanofsky/bitcoin…)). This change is just a refactoring and could be a followup.
> Perhaps we could move named argument handling and string to json conversion server side?
I think moving logic server side would avoid need for duplicate tables, but not actually make the code or logic simpler because the current syntax for distinguishing named parameters is inherently ambiguous. (It probably would have been better to require named parameters to begin with `-` to avoid ambiguity. It could also be better to try to parse *every* argument as JSON and just fall back to passing strings if they are not valid JSON to avoid the need for the conversion table.) I feel like current PR just makes some small tweaks to parsing to make the current syntax work a little better, and it adds good test coverage. If we follow up this PR with b998cc52d51b48db9271fdba0bd69e9aaccb7999 or incorporate those changes here, the client code should be better documented and more maintainable too." - ryanofsky