Make File.byLineCopy usable in @safe contexts

[0xEAB]

This depends on #9035.
Fixes one of the objectives from #10605.

[dlang-bot]

Thanks for your pull request, @0xEAB!

Bugzilla references

Your PR doesn't reference any Bugzilla issue.

If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog.

Testing this PR locally

If you don't have a local development environment setup, you can use Digger to test this PR:

dub run digger -- build "master + phobos#10606"

[0xEAB]

Do not bother reviewing this yet.
Needs a rebase once its “parent” PR is ready and AFAICT breaks one of the existing unittests.

[thewilsonator]

Parent PR merged.

[0xEAB]

Still WIP.
cae8117 is expected to fail buildkite as it makes @safe-ty assumptions only valid with DIP1000 enabled.

[0xEAB]

Oh nice, this got caught even by GHA tests.

[thewilsonator]

The PR and commit titles have some grammar issues but this otherwise looks good.

[0xEAB]

The PR and commit titles have some grammar issues but this otherwise looks good.

How about “Make File.byLineCopy usable in @safe contexts”?

[thewilsonator]

that works

[0xEAB]

that works

If you got something better, I’d be quite happy to use that instead, too :)

[0xEAB]

That label removal was GitHub’s lack of lost-update handling. (i.e. all I did was adding the “enhancement” label.)
@thewilsonator feel free to add it again.