there's no need to override the line decorations now that are the defaults

Switch from the brutalist approach to a prettier ones. If someone likes the old brutalist theme, it's still available as contrib/brutalist.config: $ cp contrib/brutalist.config ~/.telescope/config or $ telescope -c contrib/brutalist

don't need to evtimer_add to initialize the event.

don't consider an invalid tab id as an error. Things are asynchronous, we may issue a request but then stop it, and still get data relative to that. So, ignore the request when we get an invalid id.

My original idea was to crash when receiving data for an invalid tab. This idea just doesn't hold well in an asynchronous context. If we change page before the current tab finished loading we issue tab_stop, but even after that we still could receive a IMSG_BUF for that old tab id.

Another useful warning got thanks to -pedantic.

an empty initializer, such as uint8_t foo[] = { }; raises a warning: "use of GNU empty initializer extension" using -pedantic. This adds a dummy NUL byte that's not accounted in the len counter. So, now it produces: uint8_t foo[] = { 0x0 }; size_t foo_len = 0;