Ian Jackson

Results 174 comments of Ian Jackson

I don't think so. I think `handle_in` *would* have such problems with inference, since typically the closure would want to use `?`, leading to double-into. I suspect we just didn't...

If someone makes an MR branch, I can do experimental builds of `derive-builder` (and `derive-builder-fork-arti`) to see what the impact seems to be in practice.

> Maybe it's sufficient if the shadow-side error includes a hint to look at the shimlog. Something like "The managed process X failed to initialize correctly; check X.shimlog for details"...

Another repro: https://play.rust-lang.org/?version=stable&mode=debug&edition=2021&gist=f590ab120d35f6aa1d296e9da5ce9d0f @rustbot label +C-bug @rustbot label +p-high Not sure Rustbot will let me set the p-high label, but I feel I should try. After all this is a...

Additionally, this doesn't seem to be documented in the [release notes](https://github.com/dtolnay/syn/releases/tag/2.0.0) for syn 2, in the list of breaking changes. I see that `Spanned` is still implemented for every `quote::ToTokens`....

I proposed to unseal this in #1524. @dtolnay has rejected that MR. Accordingly, I think people who want `Spanned` for their own non-`ToTokens` types may need to invent a local...

I'm puzzled. You seem to be explaining that if there was a `peek($eof)`, the forms `peek2($eof)` and `peek3($eof)` would go wrong. But that doesn't seem to apply to `input.is_empty()`? Is...

> Personally, I'm a strong -1 on this. We should be documenting vulnerabilities, which are approximately an objective criteria with clear implications for users. Expanding our scope a) dilutes how...

Hi. Thanks for your attention. Sorry for the late reply; I was away last week. Unfortunately the standard file header text is not sufficient. The MPL explicitly says that it...

Oh I also notice that the SPDX string in `Cargo.toml` is wrong. It should be `LGPL-3.0-or-later`. See