+1<br><br><div class="gmail_quote">On Tue, Jul 1, 2008 at 5:38 PM, Melanie <<a href="mailto:melanie@t-data.com">melanie@t-data.com</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
Hi,<br>
<div class="Ih2E3d"><br>
Charles Krinke wrote:<br>
> Dear Kr:<br>
><br>
> The key issue seems to be that the file LSL_BuiltIn_Commands.cs contained all our C# subroutine implementations for the 328 LSL commands. Now that file has been copied and morphed.<br>
><br>
> So, my suggestion is to take the common parts of the existing file, and the copied and morphed file and put them into a third file. That third file might be LSL_Api.cs.<br>
><br>
> In any case, it seems to me that whatever unique parts are appropriate for the new scriptengine implementation and unique parts for the existing script engine implementation then reference the namespace of the common file, whatever we wish to call it.<br>

><br>
> In the common file are all the things like llSay(), llCeil(), llFloor() that just are wrappers around C# subroutines. In fact, 85% of the original LSL_BuiltIn_Commands.cs and the new, copied file are identical.<br>
><br>
> So, I would say, lets have an interface, or a derived class, or whatever you feel comfortable with and just move forward.<br>
><br>
> The most important thing to me in this whole controversy right now is to not have two identical copies of hundreds of little subroutines.<br>
<br>
</div>LSL_Api.cs _is_ that new file. No subclassing is needed or useful.<br>
The API is in LSL_Api.cs. It contains everything that was in<br>
LSL_BuiltIn_Commands.cs and obsoletes that file.<br>
Both engines will use the new pluggable API structure. Directly,<br>
without subclass, because that is what it was specifically designed<br>
to allow.<br>
<font color="#888888"><br>
Melanie<br>
</font><div><div></div><div class="Wj3C7c">_______________________________________________<br>
Opensim-dev mailing list<br>
<a href="mailto:Opensim-dev@lists.berlios.de">Opensim-dev@lists.berlios.de</a><br>
<a href="https://lists.berlios.de/mailman/listinfo/opensim-dev" target="_blank">https://lists.berlios.de/mailman/listinfo/opensim-dev</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>===================================<br>The wind<br>scours the earth for prayers<br>The night obscures them