ABX wrote:
> 
> > I tried it and it worked, but the speed difference is IMO too significant
> > for it to be used for everyday POV-Ray work.
> 
> http://news.povray.org/pce7rugbsntvn8aboobq3ea069erg1q7k3%404ax.com

I know you already mentioned that but this does not change anything about
what i said.  I'm quite confident there will be someone willing to make
regular compiles with the intel compiler once there is a standardized well
documented patch collection.

> > You should really release
> > the source code,
> 
> surprise at www.abx.art.pl/pov/popov :-)

Thanks.  I had a first quick look:

- renaming all file names to upper case is an *extremely* bad idea.
- all changes seem well marked by #ifdef's but most of the modifications
(counting number of blocks, not amount of code) are those preventing the
warnings.  I don't want to warm up this discussion again, but for the sake
of easy adaptation to new official releases (3.50c just came out) i think
this should be dropped in a general patch collection.
- the 'patches.h' file seems a good idea, i just would wish a short
description of the individual patches at each #define (i know it's in your
repository but that's not the same as if it comes with the package).

Christoph

-- 
POV-Ray tutorials, IsoWood include,                 
TransSkin and more: http://www.tu-bs.de/~y0013390/  
Last updated 13 Aug. 2002 _____./\/^>_*_<^\/\.______

Post a reply to this message