On 8/4/23 19:24, Colin Watson wrote: > This fixes a number of "table wider than line length minus indentation" > warnings from tbl. The table cells are too narrow for centered text to > look good, so left-align the contents of the text blocks. > > Reported-by: Lucas Nussbaum <lu...@debian.org> > Reported-at: https://bugs.debian.org/1042358 > Signed-off-by: Colin Watson <cjwat...@ubuntu.com> > --- > build-aux/extract-ofp-fields | 12 ++++++++---- > 1 file changed, 8 insertions(+), 4 deletions(-)
Hi, Colin. Thanks for the patches! I played around with this set and different formatting options and it seems like no matter what the end result doesn't look great. What worse is that trying to make tables fit the 80 column width actively makes them look way worse for all the larger widths, because we end up with line wrapping inside the columns even if we have enough space in the terminal. And even hyphen-split long names in the 'name' column. So, I'm thinking, how bad it is if we just put 'nowarn' there and keep all the tables as they were for many years now? I know, it's a GNU extension, but at least BSD version claims to simply ignore it, so that shouldn't cause any problems, IIUC. What do you think? If not going that route, then I'd suggest to at least not put lines without extra_name into a text box, so they will not get split in two lines. Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
