Skip to content

Commit

Permalink
Fix C++ build breakage caused by changed in stringify! output in ni…
Browse files Browse the repository at this point in the history
…ghtly Rust

The C++ build started failling with nightly rust:
rust-lang/rust#125174 changed the output of
strignify! to contins more spaces between tokens, which we relied on to
perform some type substitution from Rust types to C++ types, resulting
in compilation errors:

```
build/api/cpp/generated_include/slint_builtin_structs_internal.h:71:5: error: ‘Option’ does not name a type
   71 |     Option < core :: ops :: Range < i32 >> replacement_range;
      |     ^~~~~~
build/api/cpp/generated_include/slint_builtin_structs_internal.h:75:14: error: ‘core’ was not declared in this scope
   75 |     Option < core :: ops :: Range < i32 >> preedit_selection;
      |              ^~~~
```

Workaround by cleaning whitespace before matching the types.
  • Loading branch information
ogoffart committed Jun 15, 2024
1 parent 20c6393 commit 90739b1
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion api/cpp/cbindgen.rs
Original file line number Diff line number Diff line change
Expand Up @@ -141,7 +141,8 @@ fn builtin_structs(path: &Path) -> anyhow::Result<()> {
)*
$(
$(writeln!(file, " ///{}", $pri_doc)?;)*
let pri_type = match stringify!($pri_type) {
let pri_type = stringify!($pri_type).replace(' ', "");
let pri_type = match pri_type.as_str() {
"usize" => "uintptr_t",
"crate::animations::Instant" => "uint64_t",
// This shouldn't be accessed by the C++ anyway, just need to have the same ABI in a struct
Expand Down

0 comments on commit 90739b1

Please sign in to comment.