incubator-lucy-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Nick Wellnhofer <wellnho...@aevum.de>
Subject Re: [lucy-dev] New branch LUCY-215-cf-extensions
Date Sun, 25 Mar 2012 19:08:16 GMT
On 25/03/2012 20:33, Marvin Humphrey wrote:
> On Sat, Mar 24, 2012 at 4:37 AM, Nick Wellnhofer<wellnhofer@aevum.de>  wrote:
>
>> One thing is core/Lucy/Util/ToolSet.h, which is included from every
>> Clownfish .c file. I'm not sure what's the best way to handle that.
>
> For the record, I grepped trunk/clownfish for "ToolSet" and came up empty.  I
> believe it's used only in Lucy-specific hand-coded .c files in core/, not in
> CFC-autogenerated files.  Right?

Yes, it's just a utility file used in the Lucy .c files.

> Nevertheless, there is still a potential problem in that .cfh files are
> allowed to contain arbitrary C code, and this scenario could have arisen:
>
>      class Lucy::Foo::Bar {
>          ...
>      }
>
>      __C__
>      #include "Lucy/Util/ToolSet.h"
>      __END_C__
>
> To address that scenario, I believe that the best approach is to install all
> .h files in core/ along with all .cfh files underneath Clownfish/_include.

OK, makes sense.

Nick

Mime
View raw message