#17 braces missing for first element

[Christian Schuster]

During some test with TclSOAP, I got the strange result

name1 value1 name2 value2 {name1 value1 name2 value2}
{name1 value1 name2 value2}

when I rather expected something like

{name1 value1 name2 value2} {name1 value1 name2 value2}
{name1 value1 name2 value2}

The missing braces result from a call to "set" on the
first array element, thus breaking the structure of
$result in ::SOAP::parse_soap_response (SOAP.tcl). It
would be better to use "lappend", as it is done for
arrays in ::SOAP::Utils::decomposeSoap (utils.tcl)

[Christian Schuster]

Patch

[Kim Brennan]

In SOAP.tcl, line 1038 (line 55 of proc ::SOAP::parse_soap_response, and if statement exists, which causes this. The quick fix is to eliminate the if statement and just make that line "lappend result $r".

Is there any reason known NOT to do this?