[v2.0.2] FileSetAttrib error in File-Loop Topic is solved

[autoexec]

Testing code:

Code: Select all

Loop Files, A_Temp "\*.*", "F"
{
	;Try FileSetAttrib "-R", A_LoopFileFullPath ; This works fine.
	Try FileSetAttrib "-R" ; Critical Error: Invalid memory read/write.
	Catch As Err
		MsgBox Type(Err) ": " Err.Message "`n`n" A_LoopFileName,, "Iconx"
}

[iPhilip]

@autoexec, Can you point where in the Loop Files documentation it states that you can do that?

[gregster]

I guess the problem is rather FileSetAttrib. Its docs say for the second parameter:

If omitted, the current file of the innermost enclosing File-Loop will be used instead.

In contrast, FileGetAttrib() seems to work fine like that.

[iPhilip]

I guess the problem is rather FileSetAttrib. Its docs say for the second parameter:

If omitted, the current file of the innermost enclosing File-Loop will be used instead.

In contrast, FileGetAttrib() seems to work fine like that.

@gregster, Thank you. The quote is for FileGetAttrib but I get your point.

[gregster]

I guess the problem is rather FileSetAttrib. Its docs say for the second parameter:

If omitted, the current file of the innermost enclosing File-Loop will be used instead.

In contrast, FileGetAttrib() seems to work fine like that.

@gregster, Thank you. The quote is for FileGetAttrib but I get your point.

No, I have actually taken this quote from the FileSetAttrib docs (see FilePattern, the second parameter).
But FileGetAttrib contains the same sentence (for its first and only parameter, FileName).

[iPhilip]

No, I have actually taken this quote from the FileSetAttrib docs (see FilePattern, the second parameter).
But FileGetAttrib contains the same sentence (for its first and only parameter, FileName).

Thank you for pointing that out. It looks like a valid bug.

[autoexec]

Yes, the FileGetAttrib, FileGetSize, FileGetTime, FileGetTime, FileSetAttrib and FileSetTime also can be used in File-Loop without FilePattern. Only FileSetAttrib has a this problem.

[swagfag]

there, fixed
https://github.com/AutoHotkey/AutoHotkey/pull/323

[lexikos]

Fixed by v2.0.3.