Page MenuHomeFreeBSD

cdefs: Add __deprecated
ClosedPublic

Authored by imp on Jul 25 2024, 5:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 16, 12:47 PM
Unknown Object (File)
Sun, Mar 9, 6:28 PM
Unknown Object (File)
Feb 20 2025, 7:47 PM
Unknown Object (File)
Feb 14 2025, 12:12 AM
Unknown Object (File)
Jan 27 2025, 4:59 PM
Unknown Object (File)
Jan 24 2025, 5:17 PM
Unknown Object (File)
Jan 23 2025, 6:51 PM
Unknown Object (File)
Jan 15 2025, 4:42 AM

Details

Summary

Add __deprecated decorator. This is for a deprecated interface. We use
it today in our documentation, but don't actually provide it or use it.

Also, OpenZFS has a poorly-updated copy of compiler.h that we'll need
to fix upstream before this can land. Ideally, they'd be identical (and
OpenZFS would use the linuxkpi in FreeBSD version).

Sponsored by: Netflix

Diff Detail

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

Event Timeline

imp requested review of this revision.Jul 25 2024, 5:31 PM

copystr (see copy(9)) might be a good candidate for this that might justify adding it?

The tense in the commit message confused me: when the commit lands we will provide __deprecated so it won't be correct to say we don't provide it.

This revision is now accepted and ready to land.Jul 25 2024, 5:36 PM

The tense in the commit message confused me: when the commit lands we will provide __deprecated so it won't be correct to say we don't provide it.

Perhaps 's/don't actually/don't currently/' or 's/don't actually provide/haven't been providing or using it/'?

Yea, I hadn't planned on including the second paragraph...

Also, just looked at Linux. It defines this as just a decorator. It used to provide warnings, but doesn't anymore.
So this could whine more than Linux does when we compile linuxkpi items that provide deprecated interfaces.

Also also: it can take an arg that's a message, so maybe we want to include that.

This revision was automatically updated to reflect the committed changes.