ko1 / rubyhackchallenge

475 stars 83 forks source link

help に --dump オプションを追加します #16

Open sonots opened 7 years ago

sonots commented 7 years ago

https://bugs.ruby-lang.org/issues/13849

sonots commented 7 years ago

help をやたらと長くはしたくないそうなので、仕様議論中

sonots commented 7 years ago

https://github.com/ruby/ruby/pull/1688

--dump=? を追加するのが難しい。現状、--dump=show_help で出せるようにはなった。

$ ./ruby -h
Usage: ./ruby [switches] [--] [programfile] [arguments]
  ...
  --dump=[target] try --dump=? to show valid targets
$ ./ruby --dump=show_help
$ ./ruby --dump=show_help
Usage: ./ruby [switches] [--] [programfile] [arguments]
  --dump=insns    dump YARV instructions
  --dump=yydebug  dump yydebug of yacc parser generator
  --dump=parsetree, --dump=parsetree_with_comment
                  dump parsetree or parsetree with comment
ko1 commented 7 years ago

複数指定できることを言った方がいいかも?

sonots commented 7 years ago

--dump=insns,help とすると、help だけ出て終わったりするので、明言できないんすよねぇ。。

sonots commented 7 years ago

help は例外なので無視して、複数指定できる旨を記述します。

sonots commented 7 years ago

こうなりました

# -h では出ない
$ ./ruby --help
Usage: ./ruby [switches] [--] [programfile] [arguments]
   ...
  --dump=target[,....]
                  try --dump=help to show help messages of --dump
$ ./ruby --dump=help
Usage: ./ruby [switches] [--] [programfile] [arguments]
  --dump=insns    dump instruction sequences
  --dump=yydebug  dump yydebug of yacc parser generator
  --dump=parsetree, --dump=parsetree_with_comment
                  dump AST or AST with comments.
sonots commented 7 years ago

nobu says

What about --help=dump (--help=feature and so on)?

ってことで --help=dump に変わりました。

$ ./ruby --help
Usage: ./ruby [switches] [--] [programfile] [arguments]
   ...
  --dump=target[,....]
                  try --help=dump to show help messages of --dump
$ ./ruby --help=dump
Usage: ./ruby [switches] [--] [programfile] [arguments]
  --dump=insns    dump instruction sequences
  --dump=yydebug  dump yydebug of yacc parser generator
  --dump=parsetree, --dump=parsetree_with_comment
                  dump AST or AST with comments.
ko1 commented 7 years ago

スゴーイ。