<div dir="ltr">Are you complaining because they didn't let you saddle me with useless code?<div><br></div><div>If the code has a purpose others might be interested in the be explicit in saying so, but " Suggestions, before ripping code out, it</div>
should be better understood what you're ripping out and why. :)": is reason enough right there for ripping it out.<div class="gmail_extra"><br></div><div class="gmail_extra">Opensim is doing bad enough as is without letting people randomly force functionality on us.<br>
<br><div class="gmail_quote">On Sat, Jun 7, 2014 at 12:58 PM, Teravus Ovares <span dir="ltr"><<a href="mailto:teravus@gmail.com" target="_blank">teravus@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
I know this is old, however I wanted to mention that it is a SQLite<br>
implementation in managed C# code for cases where using the native DLL<br>
sqlite was impossible.   To test it properly you would have had to<br>
enable it and run the tests.    I suspect it's not really being used<br>
now except by maybe embedded users if at all so it's not that big of a<br>
deal...    No worries.<br>
<br>
That said, the 'reason' for ripping it out was simply warnings...<br>
which is no reason at all for ripping out functionality.    Which, I<br>
generally disapprove of.    Suggestions, before ripping code out, it<br>
should be better understood what you're ripping out and why. :).   Had<br>
you said the reason was 'lack of documentation', or some other valid<br>
reason for ripping out that bit of code, I would be happier :)<br>
<br>
Regards<br>
<br>
Teravus<br>
<br>
On Wed, Apr 23, 2014 at 12:32 PM, Justin Clark-Casey<br>
<<a href="mailto:jjustincc@googlemail.com">jjustincc@googlemail.com</a>> wrote:<br>
> afair, CSharpSQLite was something that Teravus was experimenting with for<br>
> SQLite support that we did not end up using. So yes, it's quite correct to<br>
> remove it.<br>
><br>
><br>
> On 22/04/14 18:28, Melanie wrote:<br>
>><br>
>> If OpenSim will compile and run on all supported platforms without<br>
>> it, then it's not needed and can be dropped. In that case the binary<br>
>> should also be removed from bin/<br>
>><br>
>> - Melanie<br>
>><br>
>> On 22/04/2014 18:55, Oren Hurvitz wrote:<br>
>>><br>
>>> I'm working on eliminating warnings. A few of the warnings had to do with<br>
>>> SQLite (the precise warnings aren't important, but they had to do with a<br>
>>> DLL that reimplements System.Func). I found that I can eliminate the<br>
>>> warnings if I remove the following lines from prebuild.xml:<br>
>>><br>
>>>        <Reference name="Community.CsharpSqlite" path="../../../bin/"/><br>
>>>        <Reference name="Community.CsharpSqlite.Sqlite"<br>
>>> path="../../../bin"/><br>
>>><br>
>>> This doesn't seem to have broken SQLite, because I ran the unit tests<br>
>>> afterwards (which use SQLite), and they still ran fine.<br>
>>><br>
>>> By the way, the second of these two lines was *already* in error: because<br>
>>> there's no backslash after "bin", the resulting reference was invalid.<br>
>>><br>
>>> Does anyone know of a reason why these references should stay?<br>
>>><br>
>>> Oren<br>
>>><br>
>>><br>
>>><br>
>>><br>
>>> _______________________________________________<br>
>>> Opensim-dev mailing list<br>
>>> <a href="mailto:Opensim-dev@opensimulator.org">Opensim-dev@opensimulator.org</a><br>
>>> <a href="http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev" target="_blank">http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev</a><br>
>><br>
>> _______________________________________________<br>
>> Opensim-dev mailing list<br>
>> <a href="mailto:Opensim-dev@opensimulator.org">Opensim-dev@opensimulator.org</a><br>
>> <a href="http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev" target="_blank">http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev</a><br>
>><br>
><br>
><br>
> --<br>
> Justin Clark-Casey (justincc)<br>
> OSVW Consulting<br>
> <a href="http://justincc.org" target="_blank">http://justincc.org</a><br>
> <a href="http://twitter.com/justincc" target="_blank">http://twitter.com/justincc</a><br>
><br>
> _______________________________________________<br>
> Opensim-dev mailing list<br>
> <a href="mailto:Opensim-dev@opensimulator.org">Opensim-dev@opensimulator.org</a><br>
> <a href="http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev" target="_blank">http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev</a><br>
_______________________________________________<br>
Opensim-dev mailing list<br>
<a href="mailto:Opensim-dev@opensimulator.org">Opensim-dev@opensimulator.org</a><br>
<a href="http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev" target="_blank">http://opensimulator.org/cgi-bin/mailman/listinfo/opensim-dev</a><br>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>No essence.  No permanence.  No perfection.  Only action.
</div></div>