Page MenuHomeFreeBSD

cdefs: Add __deprecated
ClosedPublic

Authored by imp on Jul 25 2024, 5:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Sep 26, 8:08 AM
Unknown Object (File)
Mon, Sep 22, 10:03 AM
Unknown Object (File)
Mon, Sep 22, 9:52 AM
Unknown Object (File)
Mon, Sep 22, 9:41 AM
Unknown Object (File)
Mon, Sep 22, 9:39 AM
Unknown Object (File)
Mon, Sep 22, 9:31 AM
Unknown Object (File)
Mon, Sep 22, 9:29 AM
Unknown Object (File)
Mon, Sep 22, 9:25 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.