BETA: SciTE v5x & lua Dynamic_include and "Smart" AutoComplete for Vars/UDFs/Abbrevs

[mLipok]

Today when I was working on wd_helper.au3 I got: 

Quote

! Loop maxed out on 50!!!

what this mean ?

[Jos]

It's a safety I build in not expecting to ever happen, but ensure there won't be some sort of endless loop.
Do you know how to replicate it? Guess you were working on a Variable at the time this happens....   remember which one?

[mLipok]

4 hours ago, Jos said:

Do you know how to replicate it?

I hope so.

I did screenshot.

I can try to make test tomorrow.

Edited February 28, 2023 by mLipok

[mLipok]

I just sent to you email: AutoItPortable_for_JOS__testing_LUA.zip - part 6 - ....

Let me know if you need some more details.

[Jos]

Thanks for the debug script: It is as I expected, a safeguard I had build-in, which I expected to be large enough but it isn't.  

When a $variable is checked for auto-include-addition, the script first checks whether it is defined in the current script by checking all whether a "$Variable" is part of a "Glocal/Local/Dim" statement or a line "$Variable=" is found. The assumption was that it normally would find no more than 50 occurrences of a variable in a particular script, but that isn't the case it this pretty large script. It is used 73 times so hence the warning message. I have left the check for max 50 in there for speed purposes and assume that when there is no definition for the $Variable in the first 50 occurrences, it likely isn't defined in this current script and bale the check. Changed it to only write an warning into the Log file form now on:

-10:54:57 ->AddMissingIncludes ->checkVarAlreadyDefined - Warning: Found 50+ occurences of variable $_WD_ERROR_Success, while checking for locally defined. Skip this check and assume not locally defined.

Change available in the Latest Beta version.

Jos 

Edited March 1, 2023 by Jos

[mLipok]

 using: And it looks like the issue is fixed.
Thanks.

Edited March 10, 2023 by Jos

[mLipok]

@Jos I attach testing script for this following issue.

 

Just open attached wd_demo.au3 and go to the last line.
And I hope you will see duplicated entries of _WD_GetSession like in the following screenshot

 

Example Part 7.zip

Edited March 8, 2023 by mLipok

[mLipok]

@Jos

I have a new feature request proposal.

I was trying to use CTRL+j to jump into 

 

EDIT:

can you add feature which will jump to desired include file which contains declaration of this value ?

Could you be so nice and add feature which will jump to desired include file which contains declaration of focused/selected value ?
Of course if this is possible and if you find some spare time....

Edited March 8, 2023 by mLipok

[Jos]

2 hours ago, mLipok said:

And I hope you will see duplicated entries of _WD_GetSession like in the following screenshot

I get that indeed and will look at it why this happens but i am pretty sure it is related to this error in au3check:

+>13:02:41 Starting AutoIt3Wrapper (22.611.2153.16) from:SciTE.exe (5.3.1.0)  Keyboard:00020409  OS:WIN_10/2009  CPU:X64 OS:X64  Environment(Language:0413)  CodePage:0  utf8.auto.check:4
+>         SciTEDir => C:\Program Files (x86)\AutoIt3\SciTE   UserDir => C:\Users\jvdza\AppData\Local\AutoIt v3\SciTE\AutoIt3Wrapper   SCITE_USERHOME => C:\Users\jvdza\AppData\Local\AutoIt v3\SciTE 
!->Includefile "z:\!!!_SVN_AU3\UDF_Forum\Other_Members\wd_helper.au3" not found.
!->Includefile "z:\!!!_SVN_AU3\UDF_Forum\Other_Members\wd_capabilities.au3" not found.

You get a clean au3check run?

Edited March 8, 2023 by Jos

[Jos]

1 hour ago, mLipok said:

Could you be so nice and add feature which will jump to desired include file which contains declaration of focused/selected value ?
Of course if this is possible and if you find some spare time....

Will have a loo when I have some time for a new challenge.  

Edited March 8, 2023 by Jos

[mLipok]

55 minutes ago, Jos said:

You get a clean au3check run?

afters some fixes ... yes.

Check my new attached zip

 

Example Part 7.zip

[Jos]

Think I found the reason why this happens in your case and believe it is fixed in the current Beta zip file.
Also try the Goto Func change (Ctrl+J)  which now checks for Variable or Other string and will try to find the first "$variable =" definition in the current file or included files.
Will later update the name of Ctrl+J to Jump2Definition or something.

Jos

[mLipok]

 using: 

And it looks like issue:

is fixed.

Thanks.

 

but I get such results:

Quote

>>> AutoItDynamicIncludes switched off for unsaved file.
+ 14:12:39 _WD_GetSession => Add include:<wd_core.au3>
> Added 1 include(s) at top of the script. (v Volume in drive C is OS
 Volume Serial Number is 9850-8576

 Directory of C:\AutoItPortable\App\SciTEv5\Lua

08.03.2023  14:12           110ÿ294 AutoItDynamicIncludes.lua
               1 File(s)        110ÿ294 bytes
               0 Dir(s)  129ÿ234ÿ165ÿ760 bytes free
)
 

 

 

EDIT: is this log intentionall ?

Edited March 10, 2023 by Jos

[mLipok]

8 minutes ago, Jos said:

Also try the Goto Func change (Ctrl+J)  which now checks for Variable or Other string and will try to find the first "$variable =" definition in the current file or included files.

testing CTRL+J while being focused on variable in line 1255 

 

[Jos]

5 minutes ago, mLipok said:

EDIT: is this log intentionall ?

which part? you mean this:

(v Volume in drive C is OS
 Volume Serial Number is 9850-8576

 Directory of C:\AutoItPortable\App\SciTEv5\Lua

08.03.2023  14:12           110ÿ294 AutoItDynamicIncludes.lua
               1 File(s)        110ÿ294 bytes
               0 Dir(s)  129ÿ234ÿ165ÿ760 bytes free
)

Don't think that should be there....  so what did you do this time to screw it up?  

[Jos]

9 minutes ago, mLipok said:

testing CTRL+J while being focused on variable in line 1255 

That is correct because I told you it works differently:

Quote

Also try the Goto Func change (Ctrl+J)  which now checks for Variable or Other string and will try to find the first "$variable =" definition in the current file or included files.

.. as that to me tells you something about the variable. My assumption is that one wants to know which value it is set to in general. 
I understand that Global $XYZ defines the variable but really doesn't tell you what its content is supposed to be.
It obviously also doesn't take scope into account like a Local $XYZ defines in multiple places in the same script.  Not planning to make this something complex like VSC and cover all possibilities. 

Edited March 8, 2023 by Jos

[mLipok]

12 minutes ago, Jos said:

Don't think that should be there....  so what did you do this time to screw it up?  

Nothing bad, truly.
I was just testing if this following issue still exist or was fixed.

When I hit ENTER it put all this log to the SciTE console and also script was suplemented by adding this:

#Region ; *** Dynamically added Include files ***
#include <wd_core.au3>                                       ; added:03/08/23 14:12:39
#EndRegion ; *** Dynamically added Include files ***

at the top of the script.

 

EDIT:

 

Edited March 8, 2023 by mLipok

[Jos]

I do not see that kind of message so have no idea as yet when that could have come from.

[mLipok]

Should I send you via email  a link to ZIP with my complete testing ENV ?

[Jos]

This must be an issue in the function Common:GetScriptName_DateTime() in common.lua...   
Please add that print(curfiledate) line in your version and perform the task again, then save the result from the Console into a file so I have the returned (exact) string and can play with the regex on the next line which seems to fail suddenly in your case:

--------------------------------------------------------------------------------
-- Get Calling Script Name and Filedate
--------------------------------------------------------------------------------
function Common:GetScriptName_DateTime()
    local curfilename = function()
      local str = debug.getinfo(2, "S").source:sub(2)
      return str:match("^.*/(.*).lua$") or str
    end
    ScriptName = curfilename()
    if debug.getinfo(2) and debug.getinfo(2).source then
        ScriptName = (debug.getinfo(2).source:match("^@-([^@]*)$") or debug.getinfo(2)) or ScriptName
    end
    curfiledate=self:perform_oscommand('dir "'..ScriptName..'"')
    print(curfiledate)
    curfiledate = curfiledate:gsub(".*(%d+)[-/](%d+)[-/](%d+)%s+(%d+):(%d+).*","%3%2%1 %4%5")
    return ScriptName:match("%w*%.[lL][uU][aA]"), curfiledate
end

 

Edited March 8, 2023 by Jos