Nick Coghlan <ncogh...@gmail.com> added the comment:

+1 for creating separate issues and linking them from this one - while the risk 
of breaking anything seems low, if we do cause a regression, multiple issues 
and PRs provide better traceability than one giant issue for everything.

(I'm also not aware of anyone else actively working on this since Sanyam's 
cProfile PR, so go ahead and create issues and PRs for any which you're 
interested in working on)

As far as `dis` specifically goes, while the function name in `dis` is 
"_test()" and it doesn't provide a meaningful help message, it's a genuinely 
useful CLI operation: it disassembles whatever file you provide, or `stdin` if 
you don't provide one:

    $ echo "print('Hello')" | python3 -m dis 
      1           0 LOAD_NAME                0 (print)
                  2 LOAD_CONST               0 ('Hello')
                  4 CALL_FUNCTION            1
                  6 POP_TOP
                  8 LOAD_CONST               1 (None)
                 10 RETURN_VALUE

So a `-m` option does make sense in `dis`, but it should probably be 
accompanied by some other changes as well (like a better name for the private 
function, and `--help` support).

----------

_______________________________________
Python tracker <rep...@bugs.python.org>
<https://bugs.python.org/issue9325>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to