Two windows side by side is nothing, if you ask me ;) Also,  IMHO having
function signatures longer than 80 cols is a serious problem, it needs to
be fixed, and not by widening the terminal.

-v


On Fri, May 25, 2018 at 8:24 AM Patrick Georgi <[email protected]> wrote:

> Am Fr., 25. Mai 2018 um 17:05 Uhr schrieb Vadim Bendebury <
> [email protected]>:
> > limiting width to 80 columns helps greatly when one wants to see several
> screens side by side.
>
> $ echo $COLUMNS
> 274
>
> Two terminals of 120 + metadata (linenumbers, window frames, ...) fit well,
> and that's just a plain old Full HD display.
> I heard 4K is all the rage now ;-)
>
> > And of course implicitly keeps the code cleaner - if so much indentation
> is needed, the code is likely ripe for refactoring.
> Several of our function signatures are longer than a line.
>
> Two levels of indentation (function level + if clause), plus
> 'printk(BIOS_ERROR, "\n");' is already more than half an 80 column screen.
> Breaking up messages means that they're harder to find with a git grep.
>
>
> Regards,
> Patrick
> --
> Google Germany GmbH, ABC-Str. 19, 20354 Hamburg
> Registergericht und -nummer: Hamburg, HRB 86891, Sitz der Gesellschaft:
> Hamburg
> Geschäftsführer: Paul Manicle, Halimah DeLaine Prado
>
-- 
coreboot mailing list: [email protected]
https://mail.coreboot.org/mailman/listinfo/coreboot

Reply via email to