Commit ca89e44
committed
Fix missing strbuf
In CI (and locally) we were seeing a failed assertion for the
`test_gem_remote_fetcher` test. After much debugging it was clear that
for this test the strbuf was missing on `shared` so when `orig` moved,
`shared` didn't follow`. This caused the offset to be very off.
```
Assertion Failed: string.c:1805:str_new_frozen_buffer:ofs >= 0
ruby 3.4.0dev (2024-08-09T19:42:38Z reproduction-for-s.. a5b29b5277) +MMTk(Immix) [arm64-darwin23]
```
With a `GC_ASSERT` we tracked down a missing `rb_mmtk_str_set_strbuf` in
`rb_str_tmp_frozen_no_embed_acquire`.
Fixes: mmtk#851 parent cd5e356 commit ca89e44
1 file changed
+5
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1671 | 1671 | | |
1672 | 1672 | | |
1673 | 1673 | | |
| 1674 | + | |
| 1675 | + | |
| 1676 | + | |
| 1677 | + | |
| 1678 | + | |
1674 | 1679 | | |
1675 | 1680 | | |
1676 | 1681 | | |
| |||
0 commit comments