Bug 8450 – measureTime doesn't work with unsafe template functions

cat > bug.d << CODE import std.datetime, std.stdio; void main() { auto mt = measureTime!((a) { writefln("Running tests took: %s ms", a.msecs()); })(); } CODE dmd bug ---- I only had a short look at this. I think the core issue is that std.traits.isSafe doesn't work for template functions. With v2.057 isSafe used to fail silently for template functions so the unsafe measureTime overload was chosen and everything worked accidentally. Since v2.058 isSafe will print an error but the @safe overload is selected.

This is a caused by bug# 8362 (that isSafe is totally broken), but the correct fix would be to have bug# 8138 fixed so that the @safety of the functions involved would be inferred. As it stands, attribute inferrence fails to solve the problem if all of the isSafe stuff is removed. I have a pull request which fixes bug# 8362, which should be updated later today.

Hmmmm. I don't think that untyped lambda functions _can_ be considered @safe. They need an actual type for that to be properly checkable. They're not isCallable, and since they aren't fully formed, you can't actually know whether or not it's @safe.

Commits pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/3bd83aad16049fe037b1cded8e386fb9a509b313 Fix for regression bug# 8450. This requires the fix for bug# 8362 in order to compile and work. https://github.com/D-Programming-Language/phobos/commit/494cbc52957b501899b42932827db9c5ae5df65d Merge pull request #725 from jmdavis/8450 Fix for regression bug# 8450.

Commit pushed to master at https://github.com/D-Programming-Language/phobos https://github.com/D-Programming-Language/phobos/commit/59f3a6ae6b6761d568aeaa434d3208ea60bb02d2 Added fix for issue# 8450 to changelog.