Page MenuHomeFreeBSD

tree(3): allow the compare function to return any signed type
ClosedPublic

Authored by glebius on Jul 6 2022, 9:20 AM.
Tags
None
Referenced Files
Unknown Object (File)
May 8 2026, 11:53 AM
Unknown Object (File)
May 8 2026, 11:17 AM
Unknown Object (File)
May 7 2026, 7:14 PM
Unknown Object (File)
May 6 2026, 11:00 PM
Unknown Object (File)
Apr 18 2026, 9:35 PM
Unknown Object (File)
Apr 14 2026, 12:03 PM
Unknown Object (File)
Apr 14 2026, 12:03 PM
Unknown Object (File)
Apr 8 2026, 12:44 AM
Subscribers

Details

Summary

This allows to write very short comparison function when we are
comparing just pointer values:

return ((intptr_t)((uintptr_t)a->ptr - (uintptr_t)b->ptr));

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

How portable is __typeof? Is it in a standard now, or sometime soon?

How portable is __typeof? Is it in a standard now, or sometime soon?

AFAIK, it is not part of standard, yet. The point is that tree.h already requires it for RB_PARENT :)

Point taken on __typeof.

In your example of use

return ((intptr_t)((uintptr_t)a->ptr - (uintptr_t)b->ptr));

can overflow, but (assuming that ptr points to something bigger than char)

return ((intptr_t)((uintptr_t)a->ptr/2 - (uintptr_t)b->ptr/2));

does not. Otherwise, it's okay by me.

This revision is now accepted and ready to land.Jul 6 2022, 6:36 PM

Good point. Maybe /2 of the result of the difference.

Good point. Maybe /2 of the result of the difference.

No, it needs to be done before the subtraction, as Doug wrote it.

Yep, now I also see it. Thanks!