Jonathan M Davis wrote: > On Thursday, October 25, 2012 15:59:39 Jens Mueller wrote: > > * Should URIs be only movable? Then you should add > > @disable this(this); > > I'd be shocked if that were appropriate. Disabling the init property like > that > should be done extremely rarely and only if you need to. It's problematic that > that feature exists at all.
That does not disable the init property. It does disable copying. > > * Regarding documentation. Please state whether it works in CTFE. Probably > > add some unittests. > > If it states that it works in CTFE, it needs unit tests to verify it. Right. Safer this way. > > I hope the module makes it into Phobos. I suggest std.net.uri. > > Thank you very much for contributing. > > We have std.uri already, so presumably, it would go there. I think it belongs inside std.net. Why not make it compatible to the current std.uri and add deprecated aliases? The std package is way too flat. Jens
