|
|
|
|
|
|
| |
| |
|
|
|
|
| |
| |
|
|
"Thorsten Froehlich" <tho### [at] trfde> wrote:
> ... is a great example for a design broken from
>the beginning...
Is there a way to talk about (may be even discuss :-) some design
aspects of the V4.0 without starting a flame war?
Andreas
Post a reply to this message
|
|
| |
| |
|
|
From: Thorsten Froehlich
Subject: Re: [BUG] POVRay excessive memory consumption
Date: 26 Jan 2004 12:41:10
Message: <40155136$1@news.povray.org>
|
|
|
| |
| |
|
|
In article <40153b45.617729628@news.povray.org> , kai### [at] siemenscom
(Andreas Kaiser) wrote:
>> ... is a great example for a design broken from
>>the beginning...
>
> Is there a way to talk about (may be even discuss :-) some design
> aspects of the V4.0 without starting a flame war?
No. As the status update says, we do not want to and will not enter into
any public discussion about 4.0 at this time.
Thorsten
____________________________________________________
Thorsten Froehlich
e-mail: mac### [at] povrayorg
I am a member of the POV-Ray Team.
Visit POV-Ray on the web: http://mac.povray.org
Post a reply to this message
|
|
| |
| |
|
|
From: Wolfgang Wieser
Subject: Re: [BUG] POVRay excessive memory consumption
Date: 29 Jan 2004 15:46:13
Message: <40197115@news.povray.org>
|
|
|
| |
| |
|
|
ABX wrote:
> Apart all others things mentioned in this thread I find:
>
> (d)[pRED] = (s)[pRED]
>
> much better "designed" than
>
> sizeof(COLOUR)/sizeof(float)
>
Is there a define/const int anywhere which tells me the size
of the COLOUR array?
Hard-coding the size 5 into MY code seems to be the worst solution.
Wolfgang
Post a reply to this message
|
|
| |
| |
|
|
From: Thorsten Froehlich
Subject: Re: [BUG] POVRay excessive memory consumption
Date: 29 Jan 2004 22:40:17
Message: <4019d221@news.povray.org>
|
|
|
| |
| |
|
|
In article <40197115@news.povray.org> , Wolfgang Wieser <wwi### [at] gmxde>
wrote:
> Is there a define/const int anywhere which tells me the size
> of the COLOUR array?
>
> Hard-coding the size 5 into MY code seems to be the worst solution.
Well, there is nothing in there that says it has to be a float, does it? So
this should read sizeof(COLOUR)/sizeof(COLC) if you really insist on the
rather pointless idea to assume the COLOUR could ever change its number of
componenents in 3.x code...
The "correct" way to implement you interpolation would be to follow current
implementation, which is to put the interpolation into a macro or inline
function in colour.h . This way anybody who might really want to try to
change the number of color components could easily find your code next to
the other code operating on colors.
Thorsten
____________________________________________________
Thorsten Froehlich, Duisburg, Germany
e-mail: tho### [at] trfde
Visit POV-Ray on the web: http://mac.povray.org
Post a reply to this message
|
|
| |
| |
|
|
From: Wolfgang Wieser
Subject: Re: [BUG] POVRay excessive memory consumption
Date: 31 Jan 2004 04:24:40
Message: <401b7457@news.povray.org>
|
|
|
| |
| |
|
|
Thorsten Froehlich wrote:
> In article <40197115@news.povray.org> , Wolfgang Wieser <wwi### [at] gmxde>
> wrote:
>
>> Is there a define/const int anywhere which tells me the size
>> of the COLOUR array?
>>
>> Hard-coding the size 5 into MY code seems to be the worst solution.
>
> Well, there is nothing in there that says it has to be a float, does it?
> So this should read sizeof(COLOUR)/sizeof(COLC) if you really insist on
> the rather pointless idea to assume the COLOUR could ever change its
> number of componenents in 3.x code...
>
Actually, I had in mind that color is "an array of 5 floats" and I did
not look it up to see that it is really COLC when writing the code.
But, right, COLC is certainly better.
> The "correct" way to implement you interpolation would be to follow
> current implementation, which is to put the interpolation into a macro or
> inline
> function in colour.h . This way anybody who might really want to try to
> change the number of color components could easily find your code next to
> the other code operating on colors.
>
Yes. But in order to make hand-patching future versions of POVRay more
easy for me, I decided to put as much code as possible into one extra
file.
There is no doubt that in case one actually wants to integrate this
patch into megapov or povray, some code has to be placed at different
locations.
Wolfgang
Post a reply to this message
|
|
| |
| |
|
|
|
|
| |