Ate Eskola

Results 66 comments of Ate Eskola

Cause for ICE: https://issues.dlang.org/show_bug.cgi?id=22390

`only()` should not be relied on being an array, that's an implementation detail. It recently was a private type, i just recently replaced it with a slice in name of...

> @dukc I am going to close this as the implementation of noreturn is still half baked. We can come back to this later. Okay

Tried to get this to compile. You guessed it: 1-3 more ICEs dug up from this rabbit hole. Here: https://issues.dlang.org/show_bug.cgi?id=23331

> I read the discussion on the other PR and AIUI it called for something that took `ubyte[]` and lazily produced validaded `char[]`. This doesn't seem to be it. Could...

Perhaps this should also accept `ushort` (assumed to be UTF-16) and `uint` (UTF-32)? Of course, this can be implemented later on just as well.

As Dennis mentioned, this should have documentation. Maybe a documented unit test. The one you added would serve well IMO. Otherwise LGTM.

Thanks! Yeah, as you suspected it's a compiler bug that can be abused without Phobos at all: ```D @safe unittest //this compiles { void escape (ref ubyte[] arr, ref ubyte[64]...

Didn't find an existing issue, so submitted: https://issues.dlang.org/show_bug.cgi?id=22709 I have another possible vulnerability in mind, but It's late around here so I'll test it another day. I hope I won't...