-
-
Notifications
You must be signed in to change notification settings - Fork 879
Closed
Description
Atom had a memory leak which is not desirable: #1803
I removed the memory and miri test to the parser: #2294
The change introduced some performance regressions and a unsafe
oxc/crates/oxc_span/src/atom.rs
Lines 103 to 104 in 6002560
| // SAFETY: It is unsafe to use this string after the allocator is dropped. | |
| Self(AtomImpl::Arena(unsafe { std::mem::transmute(s) })) |
The next goal is to remove this unsafe.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels