Odd AU3Check error.

[MvGulik]

You can use a "_Adlib_timer(111)" call after a "AdlibRegister('_Adlib_timer')".

But you can't use a "_Adlib_timer(111)" call before using the "AdlibRegister('_Adlib_timer')"

Odd ... O well.

#AutoIt3Wrapper_Run_Au3Check=y
#AutoIt3Wrapper_Au3Check_Stop_OnWarning=y
;~ #AutoIt3Wrapper_AU3Check_Parameters=-q -d -w 1 -w 2 -w 3 -w 4 -w 5 -w 6 -w 7
_Adlib_timer(111) ;; triggers AU3Check errors.
AdlibRegister('_Adlib_timer')
;~ _Adlib_timer(111) ;; not triggering AU3Check errors.
AdlibUnRegister('_Adlib_timer')
Func _Adlib_timer($parm1 = 1)
#forceref $parm1
EndFunc

#cs
>Running AU3Check (1.54.19.0)  from:C:\...\AutoIt3 || >Running AU3Check (1.54.22.0)  from:C:\...\AutoIt3\beta
D:\...\TEST.au3(5,29) : ERROR: _Adlib_timer() called with wrong number of args.
AdlibRegister('_Adlib_timer')
~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
D:\...\TEST.au3(4,17) : REF: definition of _Adlib_timer().
_Adlib_timer(111)
~~~~~~~~~~~~~~~~^
D:\...\TEST.au3(7,31) : ERROR: _Adlib_timer() called with wrong number of args.
AdlibUnRegister('_Adlib_timer')
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^
D:\...\TEST.au3(4,17) : REF: definition of _Adlib_timer().
_Adlib_timer(111)
~~~~~~~~~~~~~~~~^
D:\...\TEST.au3 - 2 error(s), 0 warning(s)
!>11:03:19 AU3Check ended.rc:2
#ce

(posting this resolve it for me.)

[BrewManNH]

Adlib functions can't have parameters, which I think might be the problem here.

[MvGulik]

Think your statement is ambiguous.

- The Adlib-Register part can't have any function parameter.

The function definition itself can have parameter. (That the parameters of a function are not used when a function is called by a Adlib-call (or other callbacks) is something else.)

[BrewManNH]

I started a topic on this not long ago in regards to an OnEvent function with parameters which was throwing errors because of it. Yes it will work with a function with parameters, but you will get error messages from AU3Check because of it.

[MvGulik]

Erm ... yea.

From Au3Check (code) point of view it probably makes perfect sens.

But that don't changes the fact that from AutoIt's point of view its legit code.

Lets leave it at that.

Edited December 1, 2011 by MvGulik

[BrewManNH]

The fact is, if you have an Adlib or OnEvent function with parameters, those parameters can't be used in the function, so they're basically just errors waiting to happen. You'll get an undeclared variable error if you try to use it, and if you try to declare it in the function you'll be told it's already declared. I think that AU3Check telling you that you made a mistake in your code is a good indication that you made a mistake somewhere in your code in this instance.

[Melba23]

Hi,

I am sure you all realise this, but if you want to use the same function in an Adlib and called direct from the script without causing Au3Check to throw a fit, you can do it by using a wrapper function like this:

_Adlib_timer_Basic(111) ; Call the basic function

AdlibRegister('_Adlib_timer', 1000) ; Call the wrapper function

$iBegin = TimerInit()
Do
    Sleep(10)
Until TimerDiff($iBegin) > 5000

AdlibUnRegister('_Adlib_timer')

; Wrapper function - no parameters
Func _Adlib_timer()
    _Adlib_timer_Basic()
EndFunc

; Basic function - with parameters
Func _Adlib_timer_Basic($parm1 = 1)
    ; Here is your function
    ConsoleWrite(@SEC & " - " & $parm1 & @CRLF)
EndFunc

M23

[BrewManNH]

That's what I did when I found out about the parameter issue, used another function for the OnEvent function that calls the main function.