[riot-notifications] [RIOT-OS/RIOT] sys/fmt: Added functions for printing table columns (#11692)

Marian Buschsieweke notifications at github.com
Mon Jul 8 21:33:30 CEST 2019


 @kaspar030: Any chance to get this upstream soon? I'd like to use this in writing other tests.

As now for reducing the stack requirements manually writing spaces is required anyway, I do not really like the use of `fmt_lpad()` any more. Writing spaces manually *and* using `fmt_lpad()` is kind of solving the same problem twice.

I could instead just use `fmt_u32_dec()` into the buffer and add the required spaces before writing the buffer. Also: Having a `static const` buffer of spaces would allow to write spaces in bigger chunks improving efficiency of the code. Don't you think 16 byte of ROM would be worth the effort to call `print()` only once every 16 byte, not for every byte?

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/11692#issuecomment-509361141
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190708/40bc8de5/attachment.html>


More information about the notifications mailing list