<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Mar 26, 2013, at 5:51 PM, Benjamin Poulain &lt;<a href="mailto:benjamin@webkit.org">benjamin@webkit.org</a>&gt; wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;"><div class="gmail_quote">On Tue, Mar 26, 2013 at 4:35 PM, Daniel Bratell<span class="Apple-converted-space">&nbsp;</span><span dir="ltr">&lt;<a href="mailto:bratell@opera.com" target="_blank">bratell@opera.com</a>&gt;</span><span class="Apple-converted-space">&nbsp;</span>wrote:<br><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex;">Den 2013-03-26 21:29:32 skrev Benjamin Poulain &lt;<a href="mailto:benjamin@webkit.org" target="_blank">benjamin@webkit.org</a>&gt;:<div><div class="h5"><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex;">On Tue, Mar 26, 2013 at 1:20 PM, Dirk Pranke &lt;<a href="mailto:dpranke@chromium.org" target="_blank">dpranke@chromium.org</a>&gt; wrote<br><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex;"><br>If we have consensus that we should just switch to paths relative to<br>Source (or maybe a couple different options), that would be (IMO) a big<br>win. It sounds like Daniel &amp; co. have already done the big bang conversion.<br><br></blockquote><br>I think using full path would be a serious hit regarding hackability.<br><br>I would rather spend some time tweaking my compiler to cache each directory<br>content than waste time finding where is every single header I need to<br>include.<br></blockquote><br></div></div>I guess you mean that it will be more job moving files around, but that is a rather rare operation, while reading an include directive and wondering what it's part of is rather common (both for compilers, tools and humans).<br></blockquote><div><br></div><div>My personal issue with full path is I don't think of the code base in term of files but in terms of classes.</div><div>I don't care where the files are, it is a detail.</div><div><br></div><div>The problem of moving file is also obvious.</div><div>There is already a huge cost associated with moving and renaming files (all the build systems). It is to the point that people will prefer leaving broken name rather than renaming headers. By having full path, you would increase that cost further, making refactoring even harder.</div><div>&nbsp;</div><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex;">I like the paths as a tool to indicate "module" dependencies. You can more easily see that a file depends on foo and bar (but not on fie) if you see:</blockquote><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex;"><br>#include "foo/object.h"<br>#include "foo/thing.h"<br>#include "bar/stuff.h"<br><br>That will tell you useful things, and avoid making layering violations by accident.<br></blockquote><div><br></div><div><div>I don't understand this argument.</div><div><br></div><div>We already have WTF, WebCore, WebKit and we use them as prefix when including headers.</div><div>The last problem is platform. But it should be fixed by moving it outside WebCore, not by changing everything else.</div></div><div><br></div><div>I think is already silly to have wtf/text as a weird exception.</div><div><br></div><blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex;">But I realize it's a question of style and as such there is not a right and a wrong, unless there are other factors. And here we have the seemingly heavy compilation time cost for it which I think is a strong argument against delegating the task of finding the header files to the compiler.</blockquote><div><br></div><div>Hackabilty is a project goal. Compile time is not.</div><div>If the change means people are afraid to move/rename files, it is a bad idea.</div><div><br></div><div>If such a change comes with the appropriate tooling for moving/renaming files, I am not opposed to it.</div></div></div></blockquote><div><br></div><div>I think a script to do a project-global search-and-replace of &lt;foo/Bar.h&gt; with &lt;fizz/Buzz.h&gt; would do the trick.</div><br><blockquote type="cite"><div style="letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px;"><div class="gmail_quote"><div><br></div><div>Benjamin</div></div>_______________________________________________<br>webkit-dev mailing list<br><a href="mailto:webkit-dev@lists.webkit.org">webkit-dev@lists.webkit.org</a><br><a href="https://lists.webkit.org/mailman/listinfo/webkit-dev">https://lists.webkit.org/mailman/listinfo/webkit-dev</a></div></blockquote></div><br></body></html>