Bug 8411 – core.time: No easy way to check if Duration is empty

The simplest way to check for an empty Duration seems to be: if (!duration.total!"hnsecs"()) { ... } I suggest either implementing an isEmpty property (given that there already is an isNegative property), or implement opCast!bool so that the following will work: if (!duration) { ... }

We now have Duration.zero, so if(duration != Duration.zero) will work as will if(duration != Duration.init) But there's no such thing as an "empty" duration. All durations have a value, and they aren't ranges or containers. It makes as much sense to have an isEmpty property on a Duration as it does to have it on an int.

An interval has a .empty property, though.

I have a further, more difficult use case for this: if(auto d = calculateSomeDuration()) { display(d); } This is not as easily worked around with the current API. Adding opCast!bool was easy enough: https://github.com/dlang/druntime/pull/1793

Commits pushed to master at https://github.com/dlang/druntime https://github.com/dlang/druntime/commit/1caa1fc2c98c0fe06b8440f435cbac744fe7729c Fix issue 8411 - add opCast!bool support for Duration. https://github.com/dlang/druntime/commit/827fb5ddc73e94b4a55bad0d7aabfa6401b52898 Merge pull request #1793 from schveiguy/addbooldur Fix issue 8411 - add opCast!bool support for Duration.

Commits pushed to stable at https://github.com/dlang/druntime https://github.com/dlang/druntime/commit/1caa1fc2c98c0fe06b8440f435cbac744fe7729c Fix issue 8411 - add opCast!bool support for Duration. https://github.com/dlang/druntime/commit/827fb5ddc73e94b4a55bad0d7aabfa6401b52898 Merge pull request #1793 from schveiguy/addbooldur

Commits pushed to newCTFE at https://github.com/dlang/druntime https://github.com/dlang/druntime/commit/1caa1fc2c98c0fe06b8440f435cbac744fe7729c Fix issue 8411 - add opCast!bool support for Duration. https://github.com/dlang/druntime/commit/827fb5ddc73e94b4a55bad0d7aabfa6401b52898 Merge pull request #1793 from schveiguy/addbooldur