Looks generally good. Can you add some tests with/without the -T option? There are two callers of _showpatch(). > + graphwidth = props.get('graphwidth') > + if not graphwidth: > + graphwidth = 0 Nit: can be written as `props.get('graphwidth', 0)`.