Custom Query

It appears from looking at the _IENavigate function that the return values from it don't match with the description in the help file. The function is supposed to return a value of -1 if it was successful, and 0 if there was a problem.

This section returns -1 with @error set if any errors are encountered in the _IELoadWait function and are passed from _IENavigate as a success, but with the @error value set to the error.

If $f_wait Then
	_IELoadWait($o_object)
	Return SetError(@error, 0, -1)
EndIf

In the File.au3 UDF _FileWriteFromArray, when passed a file name and not a handle, will open a file in OverWrite mode only. This means that if there's any Unicode characters in the array, they're written as question marks in the file.

$hFile = FileOpen($File, $FO_OVERWRITE)

I think that an easy fix/change for this would be to open it using 1 + 128 ($FO_OVERWRITE + $FO_UTF8) for UTF-8 with BOM or 1 + 256 ($FO_OVERWRITE + $FO_UTF8_NOBOM) to make it more compatable with Unicode.

In reference to this thread ​http://www.autoitscript.com/forum/topic/149831-help-on-listview-sorting/, I noticed that the function, _GUICtrlListView_SimpleSort, is changing the value of the variable passed to the function after it's done with sorting the listview.

What this does is it arbitrarily changes a value that a user passes it in their script, without telling them that it does so, and for no discernible reason, that I can see. It's inverting the value of the $vDescending variable, which is passed ByRef, from true to false or from false to true, after it has sorted the listview and recreated it. This, in my opinion, shouldn't be happening, because it can cause issues with code that uses this sort especially since it's not mentioned anywhere that it's doing this.

This section of code should be removed unless someone can explain why it's necessary, or why it's good practice.

If (IsArray($vDescending)) Then
	$vDescending[$iCol] = Not $b_desc
Else
	$vDescending = Not $b_desc
EndIf