Menu

#41 llRezObject implementation incorrect

v0.3
open
nobody
simulator (40)
5
2008-07-21
2008-07-21
Peter Quirk
No

Even though llRezObject is not implemented, I wish to report errors in the definition of the interface which result in incorrect error messages.

1. There is an error in the definition of llRezObject in LSL_Builtin_Commands_Interface.cs.

Firstly, the citation from the wiki is incorrect. It should read:
llRezObject( string inventory, vector pos, vector vel, rotation rot, integer param );
Note that the third parameter is the velocity of the object.
Secondly, the interface is missing a parameter. It should read:
void llRezObject(string inventory, LSL_Types.Vector3 pos, LSL_Types.Vector3 vel, LSL_Types.Quaternion rot, int param);

2. There is a corresponding error in file LSL_BaseClass.cs. The lines:
public void llRezObject(string inventory, vector pos, rotation rot, int param)
{
m_LSL_Functions.llRezObject(inventory, pos, rot, param);
}

should read
public void llRezObject(string inventory, vectr pos, vector vel, rotation rot, int param))
{
m_LSL_Functions.llRezObject(inventory, pos vel, rot, para,);
}

3. In file LSL_Builtin_Commands.cs, the procedure's arguments are:
public void llRezObject(string inventory, LSL_Types.Vector3 pos, LSL_Types.Quaternion rot, int param)

but should be:
public void llRezObject(string inventory, LSL_Types.Vector3 pos, LSL_Types.Vector3 vel, LSL_Type.Quaternion rot, int param)

Note that llRezObject is coded and ready for testing in OpenSim (as of r4430). Any chance we could add it in the next release?

Discussion


Log in to post a comment.

Monday.com Logo