You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Morning,
It looks like Arrays (std.container.array) data types will cause test to not compile because the destructor is marked with @System. The error message looks similar to this:
@safe function ...should cannot call @system destructor std.container.array.Range(Array!string).RangeT.~this
Not sure what the correct solution is since there was an effort to make the code safe, but this is a valid range construct, so technically it should be supported. I personally have found the ability to compare ranges very useful since it provides diffs and therefore I personally would prefer it support all ranges.
The text was updated successfully, but these errors were encountered:
It looks like it is a bug... All ranges should be supported. I will look over and see how this can be fixed. Maybe this can be easily fixed by making the assert function @trusted.
If I may suggest, perhaps it would be good to create tests to ensure that the asserts work for all of the built-in containers (std.container.*)? Or, at least a sufficient set. I know std.container.dlist works, but there's nothing that states that any of the other ones aren't going to cause issues.
Morning,
It looks like Arrays (std.container.array) data types will cause test to not compile because the destructor is marked with @System. The error message looks similar to this:
@safe function ...should cannot call @system destructor std.container.array.Range(Array!string).RangeT.~this
Not sure what the correct solution is since there was an effort to make the code safe, but this is a valid range construct, so technically it should be supported. I personally have found the ability to compare ranges very useful since it provides diffs and therefore I personally would prefer it support all ranges.
The text was updated successfully, but these errors were encountered: