Re: [info-performer] bug in src/libpr/C++/linmath.C

New Message Reply Date view Thread view Subject view Author view

From: Michael Logan (mlogan++at++mail.arc.nasa.gov)
Date: 05/24/2002 09:20:34


In C++, a pfMatrix is not an array of floats, its a structure/class.
One can't modify the memory unless you pass the address or
 'call by reference' (as in this case..)

So the call in pfLinMath.C (line 252)

    MakeZeroMat(nullmat);

fails to change anything.

BTW, I meant to send this to info-performer-dev++at++sgi.com but it
 bounced. Is that still active, or did I just mispell it?

ml

Tomlinson, Gordon wrote:

>Why say you need "pfMatrix&" a pfMatrix is simply an array of floats or doubles
>so you do not need "&" ??
>
>
>
>
>Regards
>
>Gordon.
>_________________________________________________________________
>Gordon Tomlinson
>
>Development Manager
>Urban Simulation Products.
>
>MultiGen-Paradigm
>"a Computer Associates Company"
>Suite 200, 16415 Addison Road, Addison, TX 75001 USA
>
>Tel: 972 380 9798 ext 309 Fax: 972 380 9468
>
>Email: gordon.tomlinson++at++ca.com
>
>
>-----Original Message-----
>From: Michael Logan [mailto:mlogan++at++mail.arc.nasa.gov]
>Sent: Thursday, May 23, 2002 6:47 PM
>To: info-performer++at++sgi.com
>Subject: [info-performer] bug in src/libpr/C++/linmath.C
>
>
>pfNote to anyone who is interested:
>
>There is a bug in /usr/share/Performer/src/pguide/libpr/C++/linmath.C
>
>On linux, this program gives errors (assertion failures), but on Irix,
>it seems to just get lucky.
>
>Routine MakeZeroMat() at line 86 takes a "pfMatrix" as input,
>but should take "pfMatrix&" instead. Otherwise it doesn't really zero
>out the matrix.
>
>
>thanks,
>
>mike
>

-- 
Michael Logan  Task Manager/Chief Programmer, Visual Cueing & Simulation
Raytheon ITSS  NASA Ames Research Center,MS 262-6,Moffett Field,CA, 94035
(650)-604-4494 private: vonlogan++at++rcn.com
http://users.rcn.com/vonlogan/index.htm


New Message Reply Date view Thread view Subject view Author view

This archive was generated by hypermail 2b29 : Fri May 24 2002 - 09:20:30 PDT

This message has been cleansed for anti-spam protection. Replace '++at++' in any mail addresses with the '@' symbol.