Fix overly restrictive lifetime in core::panic::Location::file return type

[ijchen]

Fixes #131770 by relaxing the lifetime to match what's stored in the struct. See that issue for more details and discussion.

Since this is a breaking change, I think a crater run is in order. Since this change should only have an effect at compile-time, I think just a check run is sufficient.

[rustbot]

r? @cuviper

rustbot has assigned @cuviper.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

[SpriteOvO]

Maybe adding a regression test in library/core/tests/panic/location.rs would be nice.

[ijchen]

Maybe adding a regression test in library/core/tests/panic/location.rs would be nice.

That's a great idea! I'll implement that.

UPDATE: Implemented. It's a bit of a strange test, since it fails through a compiler error instead of a runtime panic, but I think that's necessary considering lifetimes and the borrow checker only exists at compile time.

[cuviper]

Hmm, this is "overly restrictive" in the current implementation, but sometimes that's on purpose to give implementation freedom. For instance, we couldn't use Cow<'a, str> internally after your change. (ignoring core/alloc for a moment)

The breaking change of the function type is unlikely to be a problem IMO, but the API team should review it overall.

@rustbot label -T-libs +T-libs-api
r? libs-api

[bors]

☔ The latest upstream changes (presumably #135937) made this pull request unmergeable. Please resolve the merge conflicts.