[PATCH] lib/tevent: remove spectacularly complicated manual subtraction
Rusty Russell
rusty at rustcorp.com.au
Thu Aug 27 04:27:18 MDT 2009
On Thu, 27 Aug 2009 11:37:46 am tridge at samba.org wrote:
> In Samba we have historically not assumed that uint32_t is exactly 32
> bits, instead we have assumed it is at least 32 bits.
I realize this is now ancient history, but you really should have called it
something else then. uint32_t is defined to be 32 bits in ISO C, and so it's
amazingly confusing for newcomers like me!
I know you guys love libreplace, but there are some lines you just can't
sanely cross, like replacing pread and pwrite with seek & read/write. You've
complained about glibc doing similar to pselect, IIRC :)
> so in fact your patch would be fine with current lib/replace :-)
I *did* actually write a test program to make sure I wasn't missing some
horrible subtlety. Esp. when I saw you were the original author :)
Thanks!
Rusty.
More information about the samba-technical
mailing list