Re: [PATCH v8 07/11] iotests: limit line length to 79 chars

2020-03-24 Thread Max Reitz
On 24.03.20 18:09, John Snow wrote: > > > On 3/24/20 11:12 AM, Max Reitz wrote: >> On 24.03.20 16:02, Max Reitz wrote: >>> On 17.03.20 01:41, John Snow wrote: 79 is the PEP8 recommendation. This recommendation works well for reading patch diffs in TUI email clients. >>> >>> Also for my

Re: [PATCH v8 07/11] iotests: limit line length to 79 chars

2020-03-24 Thread John Snow
On 3/24/20 11:12 AM, Max Reitz wrote: > On 24.03.20 16:02, Max Reitz wrote: >> On 17.03.20 01:41, John Snow wrote: >>> 79 is the PEP8 recommendation. This recommendation works well for >>> reading patch diffs in TUI email clients. >> >> Also for my very GUI-y diff program (kompare). >> >>> Signe

Re: [PATCH v8 07/11] iotests: limit line length to 79 chars

2020-03-24 Thread Max Reitz
On 24.03.20 16:02, Max Reitz wrote: > On 17.03.20 01:41, John Snow wrote: >> 79 is the PEP8 recommendation. This recommendation works well for >> reading patch diffs in TUI email clients. > > Also for my very GUI-y diff program (kompare). > >> Signed-off-by: John Snow >> --- >> tests/qemu-iotes

Re: [PATCH v8 07/11] iotests: limit line length to 79 chars

2020-03-24 Thread Max Reitz
On 17.03.20 01:41, John Snow wrote: > 79 is the PEP8 recommendation. This recommendation works well for > reading patch diffs in TUI email clients. Also for my very GUI-y diff program (kompare). > Signed-off-by: John Snow > --- > tests/qemu-iotests/iotests.py | 64 +++---

Re: [PATCH v8 07/11] iotests: limit line length to 79 chars

2020-03-17 Thread John Snow
On 3/17/20 6:36 AM, Philippe Mathieu-Daudé wrote: > On 3/17/20 1:41 AM, John Snow wrote: >> 79 is the PEP8 recommendation. This recommendation works well for >> reading patch diffs in TUI email clients. >> >> Signed-off-by: John Snow >> --- >>   tests/qemu-iotests/iotests.py | 64 ++

Re: [PATCH v8 07/11] iotests: limit line length to 79 chars

2020-03-17 Thread Philippe Mathieu-Daudé
On 3/17/20 1:41 AM, John Snow wrote: 79 is the PEP8 recommendation. This recommendation works well for reading patch diffs in TUI email clients. Signed-off-by: John Snow --- tests/qemu-iotests/iotests.py | 64 +++ tests/qemu-iotests/pylintrc | 6 +++- 2 fi

[PATCH v8 07/11] iotests: limit line length to 79 chars

2020-03-16 Thread John Snow
79 is the PEP8 recommendation. This recommendation works well for reading patch diffs in TUI email clients. Signed-off-by: John Snow --- tests/qemu-iotests/iotests.py | 64 +++ tests/qemu-iotests/pylintrc | 6 +++- 2 files changed, 47 insertions(+), 23 deletion