Uploaded a new version of the SciTE4AutoIt3.exe v21.316.1639.1

[Jos]

11 minutes ago, TheDcoder said:

This is my personal opinion, but I think this is bad design, it should display all languages that are supported, otherwise it is misleading.

..and that is fine you have that opinion, but it's not going to happen on this installer.
I assume you have made the change for yourself first, before posting, and added all and think it looks great..right?

 

Edited April 27, 2021 by Jos

[TheDcoder]

2 minutes ago, Jos said:

I assume you have made the change for yourself first, before posting, and added all and think it looks great..right?

Not really, this was on a client's remote server so I didn't bother, the work will be over soon.

If I was still using Windows, I would've

[argumentum]

16 minutes ago, TheDcoder said:

my personal opinion, but I think this is bad design, it should display all languages

if you did show all the languages, you know that is not a good thing. Just add what you use. Is better

[Jos]

6 minutes ago, TheDcoder said:

Not really, this was on a client's remote server so I didn't bother, the work will be over soon.

Maybe as an suggestion: abstain from an opinion when you don't have all information to form that opinion the next time.   

[TheDcoder]

3 minutes ago, argumentum said:

If you did show all the languages, you know that is not a good thing.

I don't follow your logic, how is that a bad thing?

3 minutes ago, argumentum said:

Just add what you use. Is better

I'd be inclined to agree, but only if there was a way to configure it in the GUI... modification of the config file directly should be restricted to new experimental features or advanced features which only a few users would use.

[TheDcoder]

1 minute ago, Jos said:

abstain from an opinion when you don't have all information to form that opinion the next time.   

I am sorry Jos, but maybe you have misunderstood me, what made you think I don't have the required information to form an opinion about this?

[Jos]

Just now, TheDcoder said:

what made you think I don't have the required information to form an opinion about this?

That's easy... You state you didn't add all the possible languages:

12 minutes ago, TheDcoder said:

Not really, this was on a client's remote server so I didn't bother, the work will be over soon.

 ...but do what you feel is right for you as I have done for this installer and there is no need to agree on it.  

 

 

[TheDcoder]

1 minute ago, Jos said:

That's easy... You state you didn't add all the possible languages:

Hmmm... it's debatable I guess, but I have used other editors which provide this feature... Geany goes as far as listing all supported languages (which are many) and organizing them into their own little sub-menus. Thought that was enough to know what it would feel like in SciTE, it's only a single addition after all.

2 minutes ago, Jos said:

 ...but do what you feel is right for you as I have done for this installer and there is no need to agree on it.  

For sure, just a suggestion

[argumentum]

24 minutes ago, TheDcoder said:

should be restricted to new experimental features

H&ypertext|html|$(keyHTML)|\
#&InnoSetup|iss||\
#&Java|java||\
Java&Script|js||\
#&Kix|kix||\
#Lisp|lisp||\
#Lot|lot||\
#Lout|lt||\
Lu&a|lua||\
#Matlab|m.matlab||\

So, a coder, "DCoder", can't find his way around a configuration file ?
What would be cool, is for someone ( you ), to write a "patch", to enable/disable, and add whatever is needed for those that would like to use JS in our AU3 editor

Challenge accepted ?.

[TheDcoder]

@argumentum Too easy, I plan to make my own editor with all sorts of features... but I need to finish working on ECI before that, challenge accepted

[Trong]

Where to download the beta v21.316.1639.5 ? @Jos 

[Jos]

10 minutes ago, VIP said:

Where to download the beta v21.316.1639.5 ?

Why this question as that is now this production, so isn't available anymore?

[Trong]

I don't see it in the beta directory!
Current product version is only v21.316.1639.0, not v21.316.1639.5.

[Jos]

 

Ah, i guess I understand now: this is the published history.html in Beta:

======================= Start Updates Available in Beta ====================
14-4-2021

*** Updated Au3Stripper v21.316.1639.0.0 (Jos)
   - v21.316.1639.1  Fixed: Skip merging Directive lines ending with continuation character (9/04/2021)

*** Updated AutoIt3Wrapper v21.316.1639.0.0 (Jos)
   - v21.316.1639.2  Added: SciTE console message from Watcher when AutoIt3Wrapper was terminated. (14/04/2021)

*** Updated Tidy v21.316.1639.0.0 (Jos)
   - v21.316.1639.2  Fixed: hardcrash for one line input script (30/03/2021)
   - v21.316.1639.3  Fixed: hardcrash when option properconstants=1 is enabled (30/03/2021)
   - v21.316.1639.4  Fixed: hardcrash on a long inputline without spaces. (8/04/2021)
   - v21.316.1639.5  Fixed: Directive line ending with Underscore: space was added (9/04/2021)

======================= End Updates Available in Beta ====================

Which means that there are beta versions for the mentioned programs.
So  for example: the available Beta version of Tidy is v21.316.1639.5  and for au3stripper is v21.316.1639.1

All clear?

Jos

[RTFC]

Not sure if this is the appropriate place to report a bug in AutoIt3Wrapper (does it have its own tickets?), but line 351 in the latest version invariably crashes for me on x64 when allocating large amounts (64GB) of (physically available) RAM, as the validity of $aData is never checked before processing it as a valid 2D array (the loop needs to be conditional upon _WinAPI_EnumProcessWindows returning no @error and IsArray($aData)=True and Ubound($aData, 0)=2.

; Clean up in case the original AutoIt3Wrapper termintated but AutoIt3 is still running
            If ProcessExists($H_au3) Then
                ; first try closing nicely by sending WinClose to the windows of the PID of the running script
                Global $aData = _WinAPI_EnumProcessWindows($H_au3, False)
                For $x = 1 To $aData[0][0]  << ERROR: Subscript used on non-accessible variable
                    WinClose($aData[$x][0])
                Next

No idea why a memory squeeze might affect _WinAPI_EnumProcessWindows...

[mLipok]

This is good place do discuss.

As to reporting you can create ticket selecting SciTE4AutoIt as a Component.
But do not know what @Jos prefers.

 

Edited July 8, 2021 by mLipok
wording

[Jos]

On 7/8/2021 at 10:53 AM, RTFC said:

but line 351 in the latest version invariably crashes for me

Latest version of what exactly? AutoIt3 I assume?
Maybe an AutoIt3 bug as Autoit3Wrapper is just that...a script.  

Edited August 31, 2021 by Jos

[RTFC]

1 hour ago, Jos said:

Latest version of what

SciTE4AutoIt3 v21.316.1639.1

Sorry for being unclear. I was referring to line 351 of AutoItWrapper.au3 (as quoted), where $aData is used as a 2D array without any checks that is is actually valid. In my environment that WinAPI call fails, causing AutoItWrapper to crash.

$aData = _WinAPI_EnumProcessWindows

[Jos]

Ah ok ...  Thanks.  

I don't think anything changed in that part of the code for quite a while when it was introduced in 2019-11-17, but agree it should test for success. 
So you are proposing to add an If like this around it to fix that?

Global $aData = _WinAPI_EnumProcessWindows($H_au3, False)
                SciTE_SendConsoleMessage("> Trying normal script close by Sending WinClose to all Windows of Pid " & $H_au3)
                If IsArray($aData) Then
                    For $x = 1 To $aData[0][0]
                        WinClose($aData[$x][0])
                    Next
                EndIf

Jos

Edited July 8, 2021 by Jos

[RTFC]

Yap, or maybe (overcautious?):

If ((Not @error) And (IsArray($aData)=True) And (Ubound($aData,0)=2) Then