
Personally, I'm all for unix style bang notation/magic numbers. They start with a comment character so are ignored anywhere they aren't meaningful, and leave the door open to unanticpated future script engine shenanigans :)<div>

<br></div><div>Just my 0.02$O</div><div><br></div><div>Cheers</div><div>James</div><div><br><br><div class="gmail_quote">On Thu, Jun 28, 2012 at 4:42 PM, Justin Clark-Casey <span dir="ltr"><<a href="mailto:jjustincc@googlemail.com" target="_blank">jjustincc@googlemail.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I have made a refinement in commit 972b0b5 to still warn if no explicit language was set. 狢 believe the false positives will be lower here, since lines are less likely to end with colons for completely separate reasons.<br>


<br>

This is about balancing the known false positives against any use of this alternative engine specification facility. This problem (false positives because of : in the first line) has come up a lot in the past as well as now. 𤪻n the other hand. are you (or anybody else on this list) aware of any current cases of people using this ability to specify a different script engine? 狢'm not currently aware of any.<br>


<br>

The only other current script engine implementations I'm aware of are apart from XEngine are DNE (long discarded from core), ForthMinus (abandoned by Alondira over 4 years ago), some other DNE derived engine that was long abandoned and your own proprietary engine. 狢 don't count engines running on forked codebases since they have moved away from OpenSimulator compatibility.<br>


<br>

To be honest, I think this is an over-specified feature that was only brought in originally to avoid making a decision about OpenSimulator's script engine. 狢 find it hard to believe that anyone would want to deal with the complexity of running some scripts in one engine and other scripts in another in the same simulator, just for some theoretical performance advantage. 狢 rather doubt you're running 2 engines for the same language since that would be a nightmare to explain to users (happy to be corrected on this point).<br>


<br>

Really, I think we should adopt a better way of specifying the script language/engine than looking for a : than be stuck with this issue forever.<div class="HOEnZb"><div class="h5"><br>

<br>

On 28/06/12 02:50, Melanie wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

I'm afraid that can't be done this way. Selecting an engine without<br>

giving it an explicit language, e.g. // SCEngine: is legal.<br>

Therefore, with this patch, the defaulting to XEngine is no longer<br>

reported int these cases. Since the reporting was introduced to<br>

ensure that scripts are not run on engines with lesser<br>

functionality, the entire purpose of the message is completely<br>

defeated by this. With this patch, we might as well not notify users<br>

at all and just let them crash and burn.<br>

<br>

Melanie<br>

<br>

On 28/06/2012 02:49, Justin Clark-Casey wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

Okay Edmund, commit 25baa2d contains a change which should eliminate reporting of these false positives. 蘯lease give it<br>

a whirl.<br>

<br>

On 26/06/12 01:38, Edmund Edgar wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

On 26 June 2012 08:58, Justin Clark-Casey <<a href="mailto:jjustincc@googlemail.com" target="_blank">jjustincc@googlemail.com</a>> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

I don't like the way script language/desired engine detection was done - a<br>

clash with existing scripts could have been avoided by requiring some<br>

special preceeding chracters rather than simple c#, lsl:DotNetEngine, etc<br>

(like #! in shell scripts, for instance).<br>

</blockquote>

<br>

Agreed, but on the assumption that we're stuck with this convention<br>

now, would it be possible to tighten up the pattern matching a bit? It<br>

feels like the system should be able to tell the difference between<br>

//c#<br>

...and<br>

// This is a comment consisting of multiple words. What it has to do<br>

with script engines: Nothing.<br>

<br>

Just checking for a colon seems a bit broad. One of the mind-numbing<br>

busywork tasks on my TODO list this morning is cleaning up some<br>

scripts somebody checked in since I last fixed our iar. Their editor -<br>

I think it's the Eclipse plugin - has helpfully added:<br>

// LSL script generated:<br>

furniture-1.0.object_<u></u>definitions.presentation_<u></u>seating.lslp Sat Nov 19<br>

14:36:29 Tokyo Standard Time 2011<br>

<br>

</blockquote>

<br>

<br>

</blockquote>

______________________________<u></u>_________________<br>

Opensim-users mailing list<br>

<a href="mailto:Opensim-users@lists.berlios.de" target="_blank">Opensim-users@lists.berlios.de</a><br>

<a href="https://lists.berlios.de/mailman/listinfo/opensim-users" target="_blank">https://lists.berlios.de/<u></u>mailman/listinfo/opensim-users</a><br>

<br>

</blockquote>

<br>

<br></div></div><div class="im HOEnZb">

-- <br>

Justin Clark-Casey (justincc)<br>

<a href="http://justincc.org/blog" target="_blank">http://justincc.org/blog</a><br>

<a href="http://twitter.com/justincc" target="_blank">http://twitter.com/justincc</a><br>

<br>

<br></div><div class="HOEnZb"><div class="h5">

______________________________<u></u>_________________<br>

Opensim-users mailing list<br>

<a href="mailto:Opensim-users@lists.berlios.de" target="_blank">Opensim-users@lists.berlios.de</a><br>

<a href="https://lists.berlios.de/mailman/listinfo/opensim-users" target="_blank">https://lists.berlios.de/<u></u>mailman/listinfo/opensim-users</a><br>

</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br>===================================<br><a href="http://simhost.com" target="_blank">http://simhost.com</a><br><a href="http://twitter.com/jstallings2" target="_blank">http://twitter.com/jstallings2</a><br>

<a href="http://www.linkedin.com/pub/5/770/a49" target="_blank">http://www.linkedin.com/pub/5/770/a49</a><br>

</div>