IsEmpty() should check for `.empty()`, not `.size() == 0`

[nabijaczleweli]

[nabijaczleweli]

Any update on this?

[joakimkarlsson]

Thanks for reporting!

Is there a practical consequence to this?

[nabijaczleweli]

Yes, inability to check for "emptiness" of things that are not containers (custom smart pointers or wrappers, for example).

[joakimkarlsson]

Yes, that makes sense. Thanks.

Right now, IsEmpty() is implemented as a special case of HasLength(). We need to promote IsEmpty() to using its own IsEmptyConstraint.

[nabijaczleweli]

Any update on this?

[sbeyer]

I just looked into it in the hope that it would be simple.

For example, C++11's std::forward_list has no size() method. I included it in sequence_container_tests.cpp and noticed that a lot more than isEmpty() uses the size() method, for example, also the All operator does. Interestingly, also the so-far compiling tests fail on std::forward_list... Very weird. I pushed the work in progress to the more-container-tests branch, if interested, but I am not going to work on it now.

[sbeyer]

Update: The before-mentioned branch is already merged and deleted; it contained only preliminary work. I am now working on the issue in the issue-14 branch.