penguinolog / urwid / 36743235953

86%

master: 74%

push

github

Fix Text.pack() wrapping a line after a tab stop

Text.pack(()) started from the text width with tabs counted as 0 columns
and accepted the first layout whose lines were all narrower than that.
A wrap right after a tab stop passes that check, so
Text("Tab\there:").pack(()) returned (8, 2) for a 13-column line.

- For a StandardTextLayout the first guess counts every tab as the widest
  gap between its tab stops, so the first layout does not wrap.
- The search reads Text.layout once, so a concurrent set_layout() cannot
  switch layouts between the bound and the layout it is used with.
- A custom-layout test keeps the maxcol doubling covered.

4507 of 5466 branches covered (82.46%)

Branch coverage included in aggregate %.

9 of 9 new or added lines in 1 file covered. (100.0%)

11329 of 13017 relevant lines covered (87.03%)

9.73 hits per line