I downloaded Aladdin's Expander and managed to extract all the files,
but when trying to compile the patch I got some errors. I think I
didn't modify parse.c properly.

In particular, the section starting with:

Parse_Camera():

DJGPP says "too few arguments to function `Parse_Camera'
and also:    "parse error before `:'   "

Any help is appreciated.

Thanks,
Glen Berry

( Remove the "7" from 7no### [at] ezwvcom to email me. )

Post a reply to this message

In article <H0wNOQovLPI7G9Hdpawnc5YNnZja@4ax.com>, Glen Berry 
<7no### [at] ezwvcom> wrote:

> I downloaded Aladdin's Expander and managed to extract all the files,

Well, *some* good news. :-)


> but when trying to compile the patch I got some errors. I think I
> didn't modify parse.c properly.
> 
> In particular, the section starting with:
> 
> Parse_Camera():
> 
> DJGPP says "too few arguments to function `Parse_Camera'
> and also:    "parse error before `:'   "

You are supposed to put that part of the code in the Parse_Camera() 
function, though you can just leave it out, all it does is allow 
post_process to be specified in the camera(and currently, if you have 
some cameras defined but not used, their post_processes are still used). 
Don't put the Parse_Camera(): itself in parse.c, I was just using that 
line as a label to show where to put the following code. It might have 
helped if I had used something like "In the function Parse_Camera():" 
instead...

In the first section, labeled "tokenize.c", find the part that starts
ifdef PostProcessPatch
  {POST_PROCESS_TOKEN, "post_process"},
  {FOCAL_BLUR_TOKEN, "focal_blur"},

And add in COLOR_MATRIX_TOKEN to PATTERN_BLUR_TOKEN so that part of the 
code looks like the snippet in the file.

Do the same thing in the next section, labeled "parse.h": find the right 
spot in parse.h and copy the additional lines.
Then, in parse.c, find the function Parse_Camera(), and copy this block 
of code into the EXPECT block with all the CASE...END_CASE statements, 
right before the focal blur(although the location isn't that important, 
it seems like a good place to put it):

#ifdef PostProcessPatch
     CASE (POST_PROCESS_TOKEN)
#ifdef UnofficialBlocking
       parseUnofficialFeature(0.4);
#endif
       Parse_Post_Process();
     END_CASE
#endif

Next, delete the current Parse_Post_Process() function, and replace it 
with the given one. Replace the postproc.c and postproc.h files, and 
that should be it!

-- 
Christopher James Huff - Personal e-mail: chr### [at] yahoocom
TAG(Technical Assistance Group) e-mail: chr### [at] tagpovrayorg
Personal Web page: http://chrishuff.dhs.org/
TAG Web page: http://tag.povray.org/

Post a reply to this message

On Mon, 01 May 2000 06:39:26 -0500, Chris Huff
<chr### [at] yahoocom> wrote:


>Don't put the Parse_Camera(): itself in parse.c, I was just using that 
>line as a label to show where to put the following code.    <SNIP>

That's what I screwed up. I changed that and now DJGPP processes the
file parse.c without error. (That's the good news.)

There are some warnings issued along the way, but DJGPP seems to
continue just fine, until...

I get fatal errors during the linking stage. DJGPP returns comments
about an "  undefinded reference to `FRAND'   " (That's the bad news.)

Do I need to modify the makefile? 

(As I've said in the past, I'm really not a C programmer, as I'm sure
you can tell.)

Thanks for your help,
Glen Berry


( Remove the "7" from 7no### [at] ezwvcom to email me. )

Post a reply to this message

Glen Berry <7no### [at] ezwvcom> wrote
[ ... ]
> I get fatal errors during the linking stage. DJGPP returns comments
> about an "  undefinded reference to `FRAND'   " (That's the bad news.)
>
> Do I need to modify the makefile?
>

I haven't tried Chris's patch yet,
but FRAND() is #defined in texture.h, so maybe just adding the line
    #include "texture.h"
somewhere in the file before the offending line will do the trick.

Post a reply to this message

On Mon, 1 May 2000 11:17:23 -0400, "david sharp"
<dsh### [at] interportnet> wrote:

>I haven't tried Chris's patch yet,
>but FRAND() is #defined in texture.h, so maybe just adding the line
>    #include "texture.h" 
>somewhere in the file before the offending line will do the trick.

Thanks David,
That allowed the program to be created successfully. I've only ran one
test render so far, and haven't tried the post_process stuff yet, but
it works fine so far.

Thanks again,
Glen Berry


( Remove the "7" from 7no### [at] ezwvcom to email me. )

Post a reply to this message