Mantis Bug Tracker

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0006372opensim[REGION] Scripting Enginepublic2012-10-23 03:422012-10-26 02:55
ReporterSignpostMarv 
Assigned To 
PrioritynormalSeverityminorReproducibilitysometimes
Statuspatch includedResolutionopen 
PlatformOSOS Version
Product Version 
Target VersionFixed in Version 
Summary0006372: system ints sneaking into lists causes issues with script comms mod funcs.
DescriptionI thought I fixed this before, but it seems that mod funcs via script comms can leave system int sneaking into LSL lists, which then causes issues with ConvertFromLSL via modInvoke.
TagsNo tags attached.
Git Revision or version number
Run ModeStandalone (1 Region) , Grid (Multiple Regions per Sim)
Physics EngineBasicPhysics
Script Engine
Environment.NET / Windows32, .NET / Windows64
Mono VersionNone
Viewer
Attached Filespatch file icon modInvokeListTypes.patch [^] (3,124 bytes) 2012-10-23 03:42 [Show Content]

- Relationships

-  Notes
(0022899)
SignpostMarv (reporter)
2012-10-23 03:43

Attached patch includes an optional commit to give a consistent appearance to the if-else-if-else block.
(0022931)
justincc (administrator)
2012-10-25 17:49

Applied as git master ecdb886..01972cc. Thanks Marv.

Do you think there are similar problems with the few doubles and strings in LSL_Constants.cs? In this case, it might be easier to change those constants to be LSL wrapped classes. I did try to start this with the ints a couple of weeks ago but the sheer amount of code shuffling required (chiefly in select statements) broke me. But it might be more feasible to do this with the few strings and doubles.
(0022940)
SignpostMarv (reporter)
2012-10-26 02:55
edited on: 2012-10-26 02:57

For ints, it might not be necessary on the switch statements found in llSetPrimitiveParams, as the LSL int will happily typecast to system int.

How do switch statements work with implicit casting operators?


- Issue History
Date Modified Username Field Change
2012-10-23 03:42 SignpostMarv New Issue
2012-10-23 03:42 SignpostMarv File Added: modInvokeListTypes.patch
2012-10-23 03:43 SignpostMarv Note Added: 0022899
2012-10-23 03:43 SignpostMarv Status new => patch included
2012-10-25 17:49 justincc Note Added: 0022931
2012-10-26 02:55 SignpostMarv Note Added: 0022940
2012-10-26 02:57 SignpostMarv Note Edited: 0022940 View Revisions


Copyright © 2000 - 2012 MantisBT Group
Powered by Mantis Bugtracker