Nico Huber has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/31346 )
Change subject: libpayload/sys/types.h: Add definition for off_t ......................................................................
Patch Set 2:
But wouldn't a relation between `ssize_t` and `off_t` make it even more confusing? Generally, you don't know their size. So planting the idea that they are the same here, might con- fuse us again when we work with other definitions.
ssize_t is defined to be machine-word size (i.e. the same as uintptr_t)... at least on all systems that are and ever will be relevant to coreboot, I think. So for me that's a pretty clear one and way more tangible than "long".
I don't understand how this information can help during programming? ;) maybe when you write arch-specific code _and_ want to use an ssize_t _and_ want to be sure that more than 32 bits fit? Might be personal taste, I just wouldn't use ssize_t in such a case.