Re: Features/SourceLayout

From: Amos Jeffries <squid3@dont-contact.us>
Date: Tue, 25 Mar 2008 10:29:42 +1300

Alex Rousskov wrote:
>> The following page has been changed by Amos Jeffries:
>> http://wiki.squid-cache.org/Features/SourceLayout?action=diff&rev2=9&rev1=8
>
> Original:
>
>> Question: Can we remove Foo prefix from FOO/!FooSomething.h file
>> names? The prefix carries no additional information and is probably
>> not required for modern compilers, especially in C++ world.
>>
>> Answer: File name should match the primary class declared or defined
>> in that file. Directory name should match the namespace used by
>> classes in that directory. We should move from PROTOFoo to PROTO::Foo
>> classes.
>
> Added to Answer:
>
>> Some systemic problems have been found cleaning filenames like this
>> with compiler include methods.
>
> Amos,
>
> Please clarify what you meant. What systemic problems and with what
> kind of filenames? Is this about the case of "proto", the directory
> name? We have agreed that directories will use low_case, for Windows
> sake, right?
>

I was referring to the g++ "foo.h foo.h -I./src/Module -I./src" problem
Robert pointed out following that windows talk.
IMO the windows case-sensitivity can be worked around, but the -I
problem is a big one.

Amos

-- 
Please use Squid 2.6STABLE17+ or 3.0STABLE1+
There are serious security advisories out on all earlier releases.
Received on Mon Mar 24 2008 - 15:29:44 MDT

This archive was generated by hypermail pre-2.1.9 : Tue Apr 01 2008 - 13:00:10 MDT