Article Review: Migrating from std.date to std.datetime

[Jonathan M Davis]

On 2011-05-15 05:26, Rainer Schuetze wrote:
> Jonathan M Davis wrote:
> > On 2011-05-15 03:34, Rainer Schuetze wrote:
> >> - there is an alias for std.string.indexOf in std.datetime, that causes
> >> ambiguities when imported with other modules. (AFAICT it is used to
> >> disambiguate std.algorithm.indexOf and std.string.indexOf, which are bad
> >> by themselves, but it'd be better to use explicite std.string.indexOf in
> >> the few places in std.datetime).
> > 
> > That's a bug. I shouldn't have done that. I didn't think about the alias
> > being public when I did it. It has now been fixed on github.
> 
> Unfortunately, for some obscure reason, private module declarations are
> still placed into the overload set, so your fix does not help.
> You should either alias it to something unlikely to clash with something
> else or use fully qualified names (though this does not allow UFCS).

Bug reported: http://d.puremagic.com/issues/show_bug.cgi?id=6013

I also altered the private alias for indexOf so that it's using a name which 
won't conflict, so hopefully it won't cause any further problems, though I'd 
definitely prefer it if the bug were fixed and private alias worked properly.

- Jonathan M Davis