milksad
/
rust-bitcoin-unsafe-fast
14
0
Fork 0
Code Issues Pull Requests Activity

Document the new build script magic 

This is a follow up to #3182 which introduced a new way of
conditionally including code based on the compiler version.

When originally reviewing I missed the fact that the two loops were
controlled by the current compiler version (`minor`) so the created
macro is different dependent on the compiler used to build the code.

To help the next guy notice, add a comment.
This commit is contained in:
Tobin C. Harding 2024-08-21 17:01:41 +10:00
parent c061d936fb
commit 39f10339ad
No known key found for this signature in database
GPG Key ID: 40BF9E4C269D6607
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
internals/build.rs
View File

@ -59,6 +59,9 @@ fn write_macro(mut macro_file: impl io::Write, msrv_minor: u64, minor: u64) -> i
writeln!(macro_file, "/// Currently only the `>=` operator is supported.")?; writeln!(macro_file, "/// Currently only the `>=` operator is supported.")?;
writeln!(macro_file, "#[macro_export]")?; writeln!(macro_file, "#[macro_export]")?;
writeln!(macro_file, "macro_rules! rust_version {{")?; writeln!(macro_file, "macro_rules! rust_version {{")?;
// These two loops are the magic; we output the clause if_yes/if_no
// dependent on the current compiler version (`minor`).
for version in msrv_minor..=minor { for version in msrv_minor..=minor {
writeln!(macro_file, " (if >= 1.{} {{ $($if_yes:tt)* }} $(else {{ $($if_no:tt)* }})?) => {{", version)?; writeln!(macro_file, " (if >= 1.{} {{ $($if_yes:tt)* }} $(else {{ $($if_no:tt)* }})?) => {{", version)?;
writeln!(macro_file, " $($if_yes)*")?; writeln!(macro_file, " $($if_yes)*")?;