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)
Mar 14 2025, 12:56 AM
Unknown Object (File)
Mar 6 2025, 5:49 PM
Unknown Object (File)
Mar 6 2025, 11:24 AM
Unknown Object (File)
Mar 5 2025, 8:15 PM
Unknown Object (File)
Mar 5 2025, 2:58 AM
Unknown Object (File)
Feb 16 2025, 2:44 PM
Unknown Object (File)
Jan 30 2025, 9:37 AM
Unknown Object (File)
Jan 16 2025, 10:15 PM
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!