builtins should not print entire usage summary on invalid use #3404
Description
Consider
~ $ bind \e\c foo
Invalid token '\e\c'
fish: bind \e\c foo
^This seems like it ought to be:
~ $ bind \e\c foo
fish: bind \e\c foo
^
Invalid token '\e\c'Further:
~ $ functions -x
functions: Unknown option '-x'
functions ‐‐ print or erase functions
Synopsis
functions [ ‐a | ‐‐all ] [ ‐n | ‐‐names ]
functions ‐c OLDNAME NEWNAME
functions ‐d DESCRIPTION FUNCTION
functions [ ‐e | ‐q ] FUNCTIONS...
functions: Type 'help functions' for related documentationWe should place the error after the usage information. Sometimes it can be long and the error, far away.
5C1B