Alexander Belopolsky belopol...@users.sourceforge.net added the comment:
Reopening as a documentation issue:
- manual entry should have a versionchanged
- use *arg* instead of 'arg'
- add a ReST entry describing -m pickletools
--
components: +Documentation
nosy: +ezio.melotti
status:
Alexandre Vassalotti alexan...@peadrop.com added the comment:
LGTM
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9268
___
___
Python-bugs-list
Alexander Belopolsky belopol...@users.sourceforge.net added the comment:
Committed in r82931 with small changes based on comments here and on IRC:
1. Annotations are separated from disassembly by spaces without '|'.
2. Made a small improvement to the annotation alignment algorithm. Excessively
Changes by Alexander Belopolsky belopol...@users.sourceforge.net:
--
resolution: - fixed
stage: unit test needed - committed/rejected
status: open - closed
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9268
Antoine Pitrou pit...@free.fr added the comment:
Perhaps the annotations should be wrapped at 78 chars/line, for improved
readability?
--
___
Python tracker rep...@bugs.python.org
http://bugs.python.org/issue9268
Alexander Belopolsky belopol...@users.sourceforge.net added the comment:
Perhaps the annotations should be wrapped at 78 chars/line ..
I don't know. With deeply nested pickles, annotations may not even start
before column 80. I think further improvement in alignment and layout
algorithms
New submission from Alexander Belopolsky belopol...@users.sourceforge.net:
Example:
dis(pickle.dumps(T, 3), annotate=1)
0: \x80 PROTO 3 |