Uri class and parser

Jens Mueller jens.k.mueller at gmx.de
Thu Oct 25 14:05:14 PDT 2012


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


More information about the Digitalmars-d mailing list