Skip to content

Commit

Permalink
cleanup: Remove the const-extern-fn feature
Browse files Browse the repository at this point in the history
Since moving from a feature to a `cfg`, this feature is unneeded. Remove
it in 1.0.

Not intended for backport.
  • Loading branch information
tgross35 committed Nov 26, 2024
1 parent 093cf26 commit 369a795
Show file tree
Hide file tree
Showing 3 changed files with 2 additions and 7 deletions.
6 changes: 1 addition & 5 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -135,14 +135,10 @@ cargo-args = ["-Zbuild-std=core"]
rustc-std-workspace-core = { version = "1.0.0", optional = true }

[features]
default = ["const-extern-fn", "std"]
default = ["std"]
std = []
rustc-dep-of-std = ["rustc-std-workspace-core"]
extra_traits = []

# `const-extern-function` is deprecated and no longer does anything
# FIXME(1.0): remove this completely
const-extern-fn = []

[workspace]
members = ["libc-test"]
1 change: 0 additions & 1 deletion ci/verify-build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,6 @@ test_target() {

# Test with expected combinations of features
$cmd
$cmd --features const-extern-fn
$cmd --features extra_traits

# Test again without default features, i.e. without "std"
Expand Down
2 changes: 1 addition & 1 deletion src/macros.rs
Original file line number Diff line number Diff line change
Expand Up @@ -169,7 +169,7 @@ macro_rules! e {

// This is a pretty horrible hack to allow us to conditionally mark
// some functions as 'const', without requiring users of this macro
// to care about the "const-extern-fn" feature.
// to care about the "const-extern-fn" feature (or `const_extern_fn` config).
//
// When 'const-extern-fn' is enabled, we emit the captured 'const' keyword
// in the expanded function.
Expand Down

0 comments on commit 369a795

Please sign in to comment.