Skip to content

Commit a962605

Browse files
committed
transpile: rename minimal to conservative for --translate-const-macros
1 parent de95029 commit a962605

File tree

4 files changed

+12
-10
lines changed

4 files changed

+12
-10
lines changed

c2rust-transpile/src/lib.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,8 +49,8 @@ pub enum TranslateMacros {
4949
#[default]
5050
None,
5151

52-
/// Translate the minimal subset of macros known to always work.
53-
Minimal,
52+
/// Translate the conservative subset of macros known to always work.
53+
Conservative,
5454

5555
/// Try to translate more, but this is experimental and not guaranteed to work.
5656
///

c2rust-transpile/src/translator/mod.rs

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2166,9 +2166,11 @@ impl<'c> Translation<'c> {
21662166
let kind = &self.ast_context[expr_id].kind;
21672167
match self.tcfg.translate_const_macros {
21682168
TranslateMacros::None => Err(format_err!("translate_const_macros is None"))?,
2169-
TranslateMacros::Minimal => match *kind {
2169+
TranslateMacros::Conservative => match *kind {
21702170
CExprKind::Literal(..) => Ok(()), // Literals are leaf expressions, so they should always be const-compatible.
2171-
_ => Err(format_err!("minimal const macros don't yet allow {kind:?}"))?,
2171+
_ => Err(format_err!(
2172+
"conservative const macros don't yet allow {kind:?}"
2173+
))?,
21722174
},
21732175
TranslateMacros::Experimental => Ok(()),
21742176
}
@@ -4177,7 +4179,7 @@ impl<'c> Translation<'c> {
41774179
) -> Option<WithStmts<Box<Expr>>> {
41784180
match self.tcfg.translate_fn_macros {
41794181
TranslateMacros::None => return None,
4180-
TranslateMacros::Minimal => return None, // Nothing is supported for `Minimal` yet.
4182+
TranslateMacros::Conservative => return None, // Nothing is supported for `Conservative` yet.
41814183
TranslateMacros::Experimental => {}
41824184
}
41834185

c2rust-transpile/tests/snapshots.rs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -36,8 +36,8 @@ fn config() -> TranspilerConfig {
3636
enabled_warnings: Default::default(),
3737
emit_no_std: false,
3838
output_dir: None,
39-
translate_const_macros: TranslateMacros::Minimal,
40-
translate_fn_macros: TranslateMacros::Minimal,
39+
translate_const_macros: TranslateMacros::Conservative,
40+
translate_fn_macros: TranslateMacros::Conservative,
4141
disable_refactoring: false,
4242
preserve_unused_functions: false,
4343
log_level: log::LevelFilter::Warn,

c2rust/src/bin/c2rust-transpile.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -165,8 +165,8 @@ pub enum TranslateMacros {
165165
#[default]
166166
None,
167167

168-
/// Translate the minimal subset of macros known to always work.
169-
Minimal,
168+
/// Translate the conservative subset of macros known to always work.
169+
Conservative,
170170

171171
/// Try to translate more, but this is experimental and not guaranteed to work.
172172
///
@@ -179,7 +179,7 @@ impl From<TranslateMacros> for c2rust_transpile::TranslateMacros {
179179
fn from(this: TranslateMacros) -> Self {
180180
match this {
181181
TranslateMacros::None => c2rust_transpile::TranslateMacros::None,
182-
TranslateMacros::Minimal => c2rust_transpile::TranslateMacros::Minimal,
182+
TranslateMacros::Conservative => c2rust_transpile::TranslateMacros::Conservative,
183183
TranslateMacros::Experimental => c2rust_transpile::TranslateMacros::Experimental,
184184
}
185185
}

0 commit comments

Comments
 (0)