0002307: llParseString2List() discards initial and multiple spacers

OpenSim Mantis Tracker

Viewing Issue Simple Details [ Jump to Notes ]

[ View Advanced ] [ Issue History ] [ Print ]

Category

Severity

Reproducibility

Date Submitted

Last Update

0002307

[opensim] [REGION] Scripting Engine

minor

always

2008-10-01 06:54

2008-10-13 07:09

Reporter

Godfrey

View Status

public

Assigned To

cfk

Priority

normal

Resolution

fixed

Status

closed

Product Version

Summary

0002307: llParseString2List() discards initial and multiple spacers

Description

In Second Life's implementation of llParseString2List(), all spacer strings which occur within the source string are included in the resulting list.

In OpenSim's implementation, any spacers occurring at the beginning of the string are discarded; furthermore, if multiple spacers occur between non-spacer elements, or at the end of the source string, only the first spacer is added to the resulting list; the remainder are discarded.

In SL, the example script (provided in "Additional Information") produces the output:

Object: !, ?, !, ?, This, !, ?, is, ?, !, a, !, ?, test, ?, !, ?, !

In OpenSim, it produces this output:

Primitive: This, !, is, ?, a, !, test, ?

This issue occurs under both DotNetEngine and XEngine.

Additional Information

default
{
    state_entry()
    {
        string sourceString = "!?!?This!?is?!a!?test?!?!";
        list spacers = [ "!", "?" ];
        llOwnerSay( llList2CSV( llParseString2List( sourceString, [], spacers )));
    }
}

Tags

No tags attached.

Git Revision

SVN Revision

6603

Run Mode

Grid (1 Region per Sim) , Grid (Multiple Regions per Sim)

Physics Engine

ODE

Environment

Mono / Linux32

Mono Version

None

Attached Files

llParseString2List.patch [^] (1,546 bytes) 2008-10-02 10:37

Relationships

Notes
(0005609) Godfrey (reporter) 2008-10-02 10:02	This patch works properly for every test case I've thrown at it (testing various combinations of spacers and separators).

(0005610) nebadon (developer) 2008-10-02 10:05	i think you forgot to attach the patch :D

(0005612) Godfrey (reporter) 2008-10-02 10:37	D'OH!

(0005625) cfk (administrator) 2008-10-02 19:34	Commit as r6630

Issue History
Date Modified	Username	Field	Change
2008-10-01 06:54	Godfrey	New Issue
2008-10-01 06:54	Godfrey	SVN Revision	=> 6603
2008-10-01 06:54	Godfrey	Run Mode	=> Grid (1 Region per Sim) , Grid (Multiple Regions per Sim)
2008-10-01 06:54	Godfrey	Physics Engine	=> ODE
2008-10-01 06:54	Godfrey	Environment	=> Mono / Linux32
2008-10-01 20:07	nlin	Issue Monitored: nlin
2008-10-02 10:02	Godfrey	Note Added: 0005609
2008-10-02 10:03	Godfrey	Status	new => patch included
2008-10-02 10:05	nebadon	Note Added: 0005610
2008-10-02 10:37	Godfrey	File Added: llParseString2List.patch
2008-10-02 10:37	Godfrey	Note Added: 0005612
2008-10-02 19:34	cfk	Status	patch included => resolved
2008-10-02 19:34	cfk	Resolution	open => fixed
2008-10-02 19:34	cfk	Assigned To	=> cfk
2008-10-02 19:34	cfk	Note Added: 0005625
2008-10-13 07:09	cfk	Status	resolved => closed

Mantis 1.1.1[^]