Hi Doug, Slava,

Thanks for the suggestions. I'll implement them as soon as I can. I've
got family staying atm so it will probably be later in the week.

I'll also be rewriting alien.inline to use c-types and c-box-quot
etc. It's only after writing the initial implementation that I've become
familiar with all the applicable factor api :-)

Jeremy

Doug Coleman <[email protected]> writes:

> Jeremy,
>
> On Windows, including -fPIC in the compiler flags causes a harmless  
> warning that spams the console:
> warning: -fPIC ignored for target (all code is position independent)
>
> Otherwise, looks good!
>
> Doug
>
> On Jul 31, 2009, at 7:43 PM, Slava Pestov wrote:
>
>> Hi Jeremy,
>>
>> I noticed that changing alien.inline code and reloading the file
>> didn't make the changes take effect until Factor was restarted. Turns
>> out this was a Factor bug; add-library wasn't closing the old library
>> handle before creating the new one, and the operating system didn't
>> reload the shared object file in this case. This is fixed now.
>>
>> Also, the CM-FUNCTION: $example in the help needs to include
>> <stdlib.h>, otherwise malloc cannot be called on OpenBSD. I pushed the
>> fix for this.
>>
>> One last problem I noticed is that warnings and errors from the
>> compiler are printed to factor's error stream, and are not reported to
>> the user in any way other than that; this can be a problem when
>> running the Factor UI on Mac OS X, since the errors go to the system
>> console which I never check. How about using a process-reader to
>> collect errors, and if there are any, throw them as a Factor error?
>>
>> Finally, I'd suggest running gcc with -Werror -Wall, to catch mistakes
>> like the missing #include.
>>
>> Slava
>>
>> ------------------------------------------------------------------------------
>> Let Crystal Reports handle the reporting - Free Crystal Reports 2008  
>> 30-Day
>> trial. Simplify your report design, integration and deployment - and  
>> focus on
>> what you do best, core application coding. Discover what's new with
>> Crystal Reports now.  http://p.sf.net/sfu/bobj-july
>> _______________________________________________
>> Factor-talk mailing list
>> [email protected]
>> https://lists.sourceforge.net/lists/listinfo/factor-talk
>
>
> ------------------------------------------------------------------------------
> Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
> trial. Simplify your report design, integration and deployment - and focus on 
> what you do best, core application coding. Discover what's new with 
> Crystal Reports now.  http://p.sf.net/sfu/bobj-july
> _______________________________________________
> Factor-talk mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/factor-talk

------------------------------------------------------------------------------
Let Crystal Reports handle the reporting - Free Crystal Reports 2008 30-Day 
trial. Simplify your report design, integration and deployment - and focus on 
what you do best, core application coding. Discover what's new with 
Crystal Reports now.  http://p.sf.net/sfu/bobj-july
_______________________________________________
Factor-talk mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/factor-talk

Reply via email to