AU3Check - declared, but not used in func - what I am doing wrong ?

[mLipok]

#AutoIt3Wrapper_Au3Check_Parameters=-d -w 1 -w 2 -w 3 -w- 4 -w 5 -w 6 -w- 7

_main()
Func _main()
    Local $sSection_Function = ''
    $sSection_Function = '1'
EndFunc   ;==>_main

 

>"C:\Program Files (x86)\AutoIt3\SciTE\AutoIt3Wrapper\AutoIt3Wrapper.exe" /beta /AU3Check /in "L:Testtest_Au3Check.au3"

+>09:42:10 Starting AutoIt3Wrapper v.2.1.4.2 SciTE v.3.3.7.0 ;  Keyboard:00000415  OS:WIN_7/Service Pack 1  CPU:X64 OS:X64    Environment(Language:0415  Keyboard:00000415  OS:WIN_7/Service Pack 1  CPU:X64 OS:X64)

>Running AU3Check (3.3.11.2)  params:-d -w 1 -w 2 -w 3 -w- 4 -w 5 -w 6 -w- 7  from:C:Program Files (x86)AutoIt3Beta

"L:Testtest_Au3Check.au3"(5,31) : warning: $sSection_Function: declared, but not used in func.

Local $sSection_Function = ''

~~~~~~~~~~~~~~~~~~~~~~~~~~~~~^

L:Testtest_Au3Check.au3 - 0 error(s), 1 warning(s)

->09:42:10 AU3Check ended. Press F4 to jump to next error.rc:1

>Exit code: 0    Time: 0.548

 

 

 

Edited January 8, 2014 by mlipok

[water]

Means: You assign a value to the variable but never use the variable.

[mLipok]

Indeed, it is logical.

 

/ /_

[qwasi]

On 1/8/2014 at 12:48 PM, water said:

Means: You assign a value to the variable but never use the variable.

Hi! Sorry for bumping old threads, but what you can say in this case? 

#AutoIt3Wrapper_AU3Check_Parameters=-d -w 1 -w 2 -w 3 -w 4 -w 5 -w 6 -w 7

Opt('MustDeclareVars', 1)
__test()

Func __test()
    Local $aArray[15]
    Local $hGUI = GUICreate('', 200, 450)
    For $i = 0 To 14
    $aArray[$i] = GUICtrlCreateLabel('label_' & $i, 10, ($i * 25) + 10, 80, 21)
    Next
    Local $idBtn = GUICtrlCreateButton('Button', 10, 415, 180, 25)
    GUISetState(@SW_SHOW, $hGUI)
    While 1
        Switch GUIGetMsg()
            Case -3
                ExitLoop
            Case $idBtn
                MsgBox('64', '', $idBtn)

        EndSwitch
    WEnd
EndFunc   ;==>__test

AU3Check - $aArray: declared, but not used in func.

Ok, let's change string:

For $i = 0 To 14

to:

For $i = 0 To Ubound($aArray) - 1

 AU3Check - passed. 

imho - in this case we have a really fine line between "hold important data in variable"(control ID) and "used variable"

[water]

"Used" means: The array does not get read. Just assigning a value is not enough.

[qwasi]

2 hours ago, water said:

Just assigning a value is not enough

Ok, it's clear.  Thanks.

[jchd]

This condition isn't always strict. For instance in this code, the variable $a never gets actually used (read back at some point):

Local $a = 1

Inc($a)

Func Inc($v)
    $v += 1
EndFunc

In fact this source is equivalent to a fully empty script, doing exactly nothing except waste CPU cycles. But detecting such cases would need deep parsing, much harder to do and probably well beyond what Au3Check is aiming at.

[Jos]

1 hour ago, jchd said:

probably well beyond what Au3Check is aiming at

Yes correct...  it simply tries in a single lexing  pass to determine whether there are any obvious issues.