List of mwes that panic

[ammkrn]

List of mwes that panic

[ammkrn]

Right on. The ropey maintainer(s) agreed to accept the api extension, I'm waiting for them to review the initial implementation. I'll @ mention you if/when it gets pulled into master.

[ammkrn]

Thanks. It looks like the remaining ones are d, d, i, a, esc, b, w and i, a, esc, b, e. If the ropey maintainers are interested in having an api extension that doesn't panic, I'll see about adding one.

[archseer]

I think I addressed some of these in the latest push, it's mostly of by one errors or missing bounds checks -- I agree that having a safe indexing wrapper would be useful as well.

[archseer]

After f4560cb just dd left :)

[heliostatic]

Fantastic! Just came to report this (with backtrace) but delighted to see it will be fixed upstream.

[cessen]

Re: non-panicking version of Ropey's functions

@ammkrn and I have been discussing this in cessen/ropey#39. @archseer: if you could join the discussion over there, as the primary author of Helix, I'd really appreciate it!

[jbaa]

I believe the issue with dd was fixed with #90.

[ammkrn]

@jbaa Seems like it to me. Since the ropey api discussion is happening there, I'll go ahead and close this. Thanks!