arrayvec icon indicating copy to clipboard operation
arrayvec copied to clipboard

Consider `#[track_caller]` on functions that panic

Open joshtriplett opened this issue 3 years ago • 2 comments

For functions that panic, such as extend, please consider using #[track_caller], to provide more meaningful location information for debugging such panics.

(This report inspired by debugging an ArrayVec panic inside rust-lang/rust CI.)

joshtriplett avatar Jan 29 '22 18:01 joshtriplett

Needs knowledge about the performance impact - probably not a great concern for extend, but for other hotter methods.

bluss avatar Jan 29 '22 22:01 bluss

@bluss Absolutely. I filed this issue rather than submitting a PR, because this would absolutely require checking the performance impact.

joshtriplett avatar Jan 30 '22 01:01 joshtriplett

Fixed by #236

bluss avatar Jun 12 '23 19:06 bluss