id	summary	reporter	owner	description	type	status	milestone	component	version	severity	resolution	keywords	cc
2174	Code error in _StringInsert() - but in my opinion it should not be	amarcruz	guinness	"In Line# 270 of String.au3 (6 of _StringInsert())

{{{ElseIf $s_InsertString = """" Or (Not IsString($s_String)) Then}}}

should be read:

{{{ElseIf $s_InsertString = """" Or (Not IsString($s_InsertString)) Then}}}

by the way,
like this, many functions in Standard UDFs check its parameter type; this is NOT CONSISTENT with AutoIt philosophy, I think.

AutoIt allows expressions like this:
$x = 123 & 456      ; x=""123456""
with implicit type convertion, no error.

Why _StringInsert(""123"", 456, 3) should not working and set @error?
Why _StringInsert(""123"", """", 3) set @error?
...should not.

OK, this is my opinion, of course.
I have a 17 years base in Assembler, C, FoxPro/VFP, HTML, Javascript, and VBScript, yet I may be wrong.

...and Sorry for my horrible ""English"" :)"	Bug	closed	3.3.9.5	Standard UDFs	3.3.8.1	None	Completed	Standard UDFs, _StringInsert