Re: clisp - general interest in code cleanup

> * Daniel Jour <[email protected]> [2016-03-25 03:32:51 +0100]:
>
> Sam Steingold writes:
>> this approach does not scale.
>
> Can you elaborate on that a bit?

The approach of you refactoring many huge files and us reviewing your
changes does not scale because none of us have time for that.

>> all files include lispbibl.d, so this means that all files use them. :-)
>
> Hehe, what I meant was more like: There seems to be versions of
> alloca, an avl tree, sorting code, safe malloc and more in CLISP's code
> that could be also provided by gnulib. That's what's confusing me: Are
> these separate versions on purpose, or just not yet replaced by the
> Gnulib ones?

CLISP has a lot of code which is now available elsewhere because much of
CLISP was written 20+ years ago.

I see no problem in relying on external libraries.
I am not sure that gnulib provides avl trees though.

> What's the general course of action here? Fix it in gnulib (as far as
> that's possible, eventually using it's local modules feature) or work
> around "here"?

I suggest that you discuss this on the gnulib mailing list.
I have no idea what the gnulib people will recommend.

> What priority do non cygwin builds on windows have? Targeting mingw or
> the microsoft compilers?

Up to you - whatever compiler is easier to use/gives better speed/size
optimization &c.
I guess mingw is easier because it's the same gcc.

-- 
Sam Steingold (http://sds.podval.org/) on Ubuntu 15.10 (wily) X 11.0.11702000
http://www.childpsy.net/ http://truepeace.org http://jihadwatch.org
http://dhimmi.org http://think-israel.org http://honestreporting.com
War has never solved anything - except for ending Slavery, Nazism, Communism.

------------------------------------------------------------------------------
Transform Data into Opportunity.
Accelerate data analysis in your applications with
Intel Data Analytics Acceleration Library.
Click to learn more.
http://pubads.g.doubleclick.net/gampad/clk?id=278785351&iu=/4140
_______________________________________________
clisp-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/clisp-devel