How about after the deployment of DMD 2.049 arrange Phobos with the desired layout, then fix tool issues from using that layout.<br><br>We embrace that does not choose rule over practicality. So we can say Phobos is a flat system, but break that for good reason such as std.container and std.algorithm. How they should be broken up is another subject.<br>
<br><div class="gmail_quote">On Thu, Sep 16, 2010 at 6:52 AM, David Simcha <span dir="ltr"><<a href="mailto:dsimcha@gmail.com">dsimcha@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
I'd like to see a way to make Ddoc produce documentation for the public imports of a module. This way we could make a simple, coarse-grained import system for the user (i.e. import std.container and you've got every container you could possibly want), but have each container be implemented in its own file in std.containerimpl.somecontainer for development convenience and publicly imported by std.container. The only major problem I see with this is Ddoc, as mentioned above.<br>
<br>The bottom line is that, when using D, I hate being forced to write tons of import statement boilerplate to do anything non-trivial (what you get if you make the import system fine-grained) but when developing Phobos I hate having to scroll through 6,000 line files like std.algorithm (what you get when you make the import system coarse-grained). I'd like to clean this up from the developer's perspective, but in a way that's transparent to the user.<div>
<div></div><div class="h5"><br>
<br><div class="gmail_quote">On Thu, Sep 16, 2010 at 9:46 AM, Jacob Carlborg <span dir="ltr"><<a href="mailto:doob@me.com" target="_blank">doob@me.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
I don't think it's the number of modules that makes it hard, it's because of the organization of the modules. Too many modules in one package (std) and too much code in one module. Take std.container for example, every container type in on module, that's just insane. It should clearly be, in my opinion, a container package (or actually I would prefer it to be called collection) and one module per container type. std.container is almost 3500 LOC and have four (or something like that) container types, what happens when (if) we get more container types like different types of trees and other containers, just put everything in std.container? That is just too much code in one module. Just my opinion.<br>
<br>
<br>
/Jacob Carlborg<br>
<div><div></div><div><br>
On 16 sep 2010, at 15:30, Lars Tandle Kyllingstad wrote:<br>
<br>
> I disagree, I like the flat hierarchy.<br>
><br>
> And just to clarify my earlier e-mail, for which I may have chosen a bad<br>
> subject: I do not think that the number of modules in Phobos is too<br>
> high in general. I think it has too many *obsolete* modules, and too<br>
> many modules with very limited functionality.<br>
><br>
> If we got rid of the cruft that has been accumulating, and merge some of<br>
> the smaller modules into larger ones, Phobos would seem a lot less<br>
> unwieldly.<br>
><br>
> -Lars<br>
><br>
><br>
><br>
> On Thu, 2010-09-16 at 14:54 +0200, Jacob Carlborg wrote:<br>
>> I think it's time for Phobos to start using sub packages and drop the flat module hierarchy.<br>
>><br>
>> On 15 sep 2010, at 13:56, Lars Tandle Kyllingstad wrote:<br>
>><br>
>>> I said in an earlier e-mail that I think std.file, std.path, and<br>
>>> std.stdio should remain separate modules. However, I do think that, for<br>
>>> a library with a flat module hierarchy, Phobos has acquired way too many<br>
>>> modules. Some of them should (and will) be removed, and some could be<br>
>>> merged.<br>
>>><br>
>>> The following are my suggestions for how to trim the Phobos module list<br>
>>> a bit. At the bottom I'll show the resulting module list.<br>
>>><br>
>>><br>
>>> Modules which could be removed, most of them *right now*, because they<br>
>>> are superseded by other modules or built-in functionality:<br>
>>><br>
>>> std.bind - use lambdas or nested functions instead<br>
>>> std.boxer - superseded by std.variant<br>
>>> std.contracts - superseded by std.exception<br>
>>> std.cstream - superseded by std.stdio.File<br>
>>> std.demangle - superseded by core.demangle<br>
>>> std.iterator - superseded by std.range<br>
>>> std.openrj - obscure format, better to use std.json<br>
>>> std.perf - superseded by StopWatch<br>
>>> std.regexp - superseded by std.regex<br>
>>> std.stream - ranges are the way to go<br>
>>> std.syserror - superseded by std.windows.syserror<br>
>>> std.c.* - superseded by core.stdc.* and core.sys.*<br>
>>><br>
>>> Modules for which there is no documentation on the D home page, and<br>
>>> which I suspect nobody are using:<br>
>>><br>
>>> std.loader<br>
>>> std.stdarg<br>
>>> std.typelist<br>
>>><br>
>>> Modules which can be merged into a single one, possibly after<br>
>>> substantial/complete rewrites:<br>
>>><br>
>>> std.compiler + std.cpuid + std.system (= std.sysinfo?)<br>
>>> std.ctype + std.uni (= std.character?)<br>
>>> std.date + std.dateparse + std.gregorian + std.stopwatch<br>
>>> (= std.datetime?)<br>
>>> std.encoding + std.utf (= std.encoding?)<br>
>>> std.socket + std.socketstream (= std.socket (or <a href="http://std.net?" target="_blank">std.net?</a>))<br>
>>> std.typecons + std.typetuple (= std.types?)<br>
>>><br>
>>> And finally, some renaming suggestions (which were actually posed by<br>
>>> Andrei):<br>
>>><br>
>>> std.conv -> std.convert<br>
>>> std.stdio -> <a href="http://std.io" target="_blank">std.io</a><br>
>>><br>
>>><br>
>>> All this gives us the following, nice and short (well... shorter, at<br>
>>> least) module list:<br>
>>><br>
>>> std.algorithm<br>
>>> std.array<br>
>>> std.base64<br>
>>> std.bigint<br>
>>> std.bitmanip<br>
>>> std.character<br>
>>> std.complex<br>
>>> std.concurrency<br>
>>> std.container<br>
>>> std.convert<br>
>>> std.datetime<br>
>>> std.encoding<br>
>>> std.exception<br>
>>> std.file<br>
>>> std.format<br>
>>> std.functional<br>
>>> std.getopt<br>
>>> std.intrinsic<br>
>>> <a href="http://std.io" target="_blank">std.io</a><br>
>>> std.json<br>
>>> std.math<br>
>>> std.md5<br>
>>> std.metastrings<br>
>>> std.mmfile<br>
>>> std.numeric<br>
>>> std.outbuffer<br>
>>> std.path<br>
>>> std.process<br>
>>> std.random<br>
>>> std.range<br>
>>> std.regex<br>
>>> std.signals<br>
>>> std.socket<br>
>>> std.stdint<br>
>>> std.string<br>
>>> std.sysinfo<br>
>>> std.traits<br>
>>> std.types<br>
>>> std.variant<br>
>>> std.xml<br>
>>> std.zip<br>
>>> std.linux<br>
>>> std.windows<br>
>>><br>
>>><br>
>>> Just something to think about.<br>
>>><br>
>>> -Lars<br>
>>><br>
>>> _______________________________________________<br>
>>> phobos mailing list<br>
>>> <a href="mailto:phobos@puremagic.com" target="_blank">phobos@puremagic.com</a><br>
>>> <a href="http://lists.puremagic.com/mailman/listinfo/phobos" target="_blank">http://lists.puremagic.com/mailman/listinfo/phobos</a><br>
>><br>
><br>
><br>
> _______________________________________________<br>
> phobos mailing list<br>
> <a href="mailto:phobos@puremagic.com" target="_blank">phobos@puremagic.com</a><br>
> <a href="http://lists.puremagic.com/mailman/listinfo/phobos" target="_blank">http://lists.puremagic.com/mailman/listinfo/phobos</a><br>
<br>
</div></div>--<br>
<font color="#888888">/Jacob Carlborg<br>
</font><div><div></div><div><br>
_______________________________________________<br>
phobos mailing list<br>
<a href="mailto:phobos@puremagic.com" target="_blank">phobos@puremagic.com</a><br>
<a href="http://lists.puremagic.com/mailman/listinfo/phobos" target="_blank">http://lists.puremagic.com/mailman/listinfo/phobos</a><br>
</div></div></blockquote></div><br>
</div></div><br>_______________________________________________<br>
phobos mailing list<br>
<a href="mailto:phobos@puremagic.com">phobos@puremagic.com</a><br>
<a href="http://lists.puremagic.com/mailman/listinfo/phobos" target="_blank">http://lists.puremagic.com/mailman/listinfo/phobos</a><br></blockquote></div>