SciTe

[anixon]

I am running AutoIT in Windows Vista SP1.

I have a script which is now 2630 lines long and when running the Tools/Tidy AutoIT Scource I get the following Windows Message Box:

"Tidy your AutoIT3 Script has stopped Working"

However the Tidy process appears to complete and is reporting in the control panel:

>"C:\Program Files\AutoIt3\SciTE\tidy\tidy.exe" "C:\PhoneServer\Source Code\sSentry.au3"

Tidy AutoIt3 v2.0.23.14 Copyright © Jos van der Zande May 7, 2008

*** BACKUPDIR$:

*** TargetDir$:c:\phoneserver\source code\BackUp\

+> Tidy AutoIt3 finished. Original copied to:"c:\phoneserver\source code\BackUp\ssentry_old10.au3"

When running the Tidy process on scripts with less than 1000 lines this problem does not present.

Ant..

[Jos]

I haven't seen this before.

What happens when you run it on the AutoIt3Wrapper.au3 script which is 2300+ lines?

Jos

[anixon]

I haven't seen this before.

What happens when you run it on the AutoIt3Wrapper.au3 script which is 2300+ lines?

Jos

Well that's interesting because it does not result in the same symptom. I added 300 lines to the wrapper script but that did not produce an error so one

could conclude it is not an issue with the number of lines. I can live without a resolution as the script works fine in every other respect including obfscating and compiling

Cheers Ant..

[PsaltyDS]

Well that's interesting because it does not result in the same symptom. I added 300 lines to the wrapper script but that did not produce an error so one

could conclude it is not an issue with the number of lines. I can live without a resolution as the script works fine in every other respect including obfscating and compiling

Cheers Ant..

P.S. IE.au3 is ove 4600 lines - no problems with Tidy.

[anixon]

P.S. IE.au3 is ove 4600 lines - no problems with Tidy.

I think that it is code related and specific to my script I have run tidy on the same script but a version of several days ago and there is no problem. So I am going to go through it and see if I can identify what change to the code has generated the issue. I guess just a process of elimination.

Ant..

[anixon]

In trying to identify the issue I restored my operating system to a couple of days ago and that made no difference. Ant..

[PsaltyDS]

I think that it is code related and specific to my script I have run tidy on the same script but a version of several days ago and there is no problem. So I am going to go through it and see if I can identify what change to the code has generated the issue. I guess just a process of elimination.

Ant..

If you find something specific please post it here.

[anixon]

Ok I can now tell you what the issue is. It would appear that when using "Global" to define multiple variables over multiple lines ie

;//Global Variables:

Global $Font = "times new roman", $BeepNum, $BeepStart, $BeepStop, $PingID1, $PingIP1, $PingDelay1, $SkypeStatus, $PingID2, $PingIP2, _

$PingDelay2, $ProcessCycle, $MessageDisplay, $MessageDelay, $Delay, $WriteLog, $SoundVolume, $AppSentry1, $AppSentry2, $PingCounter, _

$File, $LogFlag, $SkypeStatusFlag, $Flag, $LicDLTR, $LMsg, $InstallLog_ss, $LicenceLog_dd, $WinInstall_ss, $Install_ss, $Install_dd, _

$Licence_dd, $ExpiryDate, $InstallDate, $sSensitivity, $SkypeMissedCalls, $SkypeMissedMsgs, $SkypeVoiceMails, $sSensitivityIsCon, _

$sSensitivityPing1, $PsSensitivityPing2, $sSensitivityPing2, $sSensitivityErr, $s_SmtpServer, $s_FromName, $s_FromAddress, $LICENCEData, _

$s_ToAddress, $s_ToAddressM, $s_ToAddressR, $s_Subject, $as_Body, $s_LogFileID, $s_CcAddress, $s_CcAddressM, $s_CcAddressR, $s_BccAddress, _

$s_Username, $s_Password, $s_IPPort, $s_ssl, $s_ReportTime, $SkypeMcFlag, $SkypeMmFlag, $SkypeVmFlag, $s_EmailAttachment, $s_SkypeMailOnOff, _

$s_ReportMailOnOff, $s_MailCheck, $s_TestTransport, $oSkypeMissedCallsName, $oSkypeMissedMsgsName, $CallFlag, $CallsText, $SoundText, _

$oSkypeMissedVoiceName, $LicenceEmail, $MonthlyLog, $DailyLog, $MissedNameCalls, $MissedNameMsgs, $MissedNameVoice, _

$MissedCallsFlag, $s_EmailCycle, $s_EMailOrSmsM, $TitleM, $s_EMailOrSmsR, $TitleR, $DailyReport, $DailyReportFile, $sSPassword, _

$ReportDayDate, $ReportMonthDate, $ReportText, $BeepFlag, $StartPing, $EndPing, $sSTrayExit, $sSTrayMenu, $sSTrayToggle, $sSTrayAlerts, _

$sSTraySound, $StartCycle, $EndCycle, $StartECycle, $EndECycle, $sSTrayStatus, $LogTextS, $LogFile, $OldHour, $ViewFile, _

$ToAddressR, $s_CallFileID, $mprocess, $TextMsg, $ipFlag, $key, $Password, $logtext, $message, $logo, $s_Editor, $s_smtpTransport, _

$mailcheck, $CallsStop, $CallsStart, $s_CallsStop, $s_CallsStart, $CallsFlag, $iRegKey1, $iRegKey2, $iRegKey3, $iRegKey4, $Display, _

$s_SendersAddress, $s_StartCode, $s_StopCode, $ProcessedCalls, $ProcessedMsgs, $ProcessedVoice, $mMailServer, $mGateway, $mRouter, _

$mSkype, $mApplic2, $MServerTxt, $GatewayTxt, $RouterTxt, $SkypeTxt, $Applic2Txt, $StatusMsgFlag

and you exceed a specific number of variables and lines Tidy will fail as it does on the above.

To fix the problem I simply split the Global statement into two to reduce the number of $variables and "_" for multiple lines for each

It now looks like this and it works....

;//Global Variables:

Global $Font = "times new roman", $BeepNum, $BeepStart, $BeepStop, $PingID1, $PingIP1, $PingDelay1, $SkypeStatus, $PingID2, $PingIP2, _

$PingDelay2, $ProcessCycle, $MessageDisplay, $MessageDelay, $Delay, $WriteLog, $SoundVolume, $AppSentry1, $AppSentry2, $PingCounter, _

$File, $LogFlag, $SkypeStatusFlag, $Flag, $LicDLTR, $LMsg, $InstallLog_ss, $LicenceLog_dd, $WinInstall_ss, $Install_ss, $Install_dd, _

$Licence_dd, $ExpiryDate, $InstallDate, $sSensitivity, $SkypeMissedCalls, $SkypeMissedMsgs, $SkypeVoiceMails, $sSensitivityIsCon, _

$sSensitivityPing1, $PsSensitivityPing2, $sSensitivityPing2, $sSensitivityErr, $s_SmtpServer, $s_FromName, $s_FromAddress, $LICENCEData, _

$s_ToAddress, $s_ToAddressM, $s_ToAddressR, $s_Subject, $as_Body, $s_LogFileID, $s_CcAddress, $s_CcAddressM, $s_CcAddressR, $s_BccAddress, _

$s_Username, $s_Password, $s_IPPort, $s_ssl, $s_ReportTime, $SkypeMcFlag, $SkypeMmFlag, $SkypeVmFlag, $s_EmailAttachment, $s_SkypeMailOnOff, _

$s_ReportMailOnOff, $s_MailCheck, $s_TestTransport, $oSkypeMissedCallsName, $oSkypeMissedMsgsName, $CallFlag, $CallsText, $SoundText

Global $oSkypeMissedVoiceName, $LicenceEmail, $MonthlyLog, $DailyLog, $MissedNameCalls, $MissedNameMsgs, $MissedNameVoice, _

$MissedCallsFlag, $s_EmailCycle, $s_EMailOrSmsM, $TitleM, $s_EMailOrSmsR, $TitleR, $DailyReport, $DailyReportFile, $sSPassword, _

$ReportDayDate, $ReportMonthDate, $ReportText, $BeepFlag, $StartPing, $EndPing, $sSTrayExit, $sSTrayMenu, $sSTrayToggle, $sSTrayAlerts, _

$sSTraySound, $StartCycle, $EndCycle, $StartECycle, $EndECycle, $sSTrayStatus, $LogTextS, $LogFile, $OldHour, $ViewFile, _

$ToAddressR, $s_CallFileID, $mprocess, $TextMsg, $ipFlag, $key, $Password, $logtext, $message, $logo, $s_Editor, $s_smtpTransport, _

$mailcheck, $CallsStop, $CallsStart, $s_CallsStop, $s_CallsStart, $CallsFlag, $iRegKey1, $iRegKey2, $iRegKey3, $iRegKey4, $Display, _

$s_SendersAddress, $s_StartCode, $s_StopCode, $ProcessedCalls, $ProcessedMsgs, $ProcessedVoice, $mMailServer, $mGateway, $mRouter, _

$mSkype, $mApplic2, $MServerTxt, $GatewayTxt, $RouterTxt, $SkypeTxt, $Applic2Txt, $StatusMsgFlag

I will leave it up to you as to whether it needs a fix.... Ant..

[PsaltyDS]

Ok I can now tell you what the issue is. It would appear that when using "Global" to define multiple variables over multiple lines ie

CODE

;//Global Variables:

Global $Font = "times new roman", $BeepNum, $BeepStart, $BeepStop, $PingID1, $PingIP1, $PingDelay1, $SkypeStatus, $PingID2, $PingIP2, _

$PingDelay2, $ProcessCycle, $MessageDisplay, $MessageDelay, $Delay, $WriteLog, $SoundVolume, $AppSentry1, $AppSentry2, $PingCounter, _

$File, $LogFlag, $SkypeStatusFlag, $Flag, $LicDLTR, $LMsg, $InstallLog_ss, $LicenceLog_dd, $WinInstall_ss, $Install_ss, $Install_dd, _

$Licence_dd, $ExpiryDate, $InstallDate, $sSensitivity, $SkypeMissedCalls, $SkypeMissedMsgs, $SkypeVoiceMails, $sSensitivityIsCon, _

$sSensitivityPing1, $PsSensitivityPing2, $sSensitivityPing2, $sSensitivityErr, $s_SmtpServer, $s_FromName, $s_FromAddress, $LICENCEData, _

$s_ToAddress, $s_ToAddressM, $s_ToAddressR, $s_Subject, $as_Body, $s_LogFileID, $s_CcAddress, $s_CcAddressM, $s_CcAddressR, $s_BccAddress, _

$s_Username, $s_Password, $s_IPPort, $s_ssl, $s_ReportTime, $SkypeMcFlag, $SkypeMmFlag, $SkypeVmFlag, $s_EmailAttachment, $s_SkypeMailOnOff, _

$s_ReportMailOnOff, $s_MailCheck, $s_TestTransport, $oSkypeMissedCallsName, $oSkypeMissedMsgsName, $CallFlag, $CallsText, $SoundText, _

$oSkypeMissedVoiceName, $LicenceEmail, $MonthlyLog, $DailyLog, $MissedNameCalls, $MissedNameMsgs, $MissedNameVoice, _

$MissedCallsFlag, $s_EmailCycle, $s_EMailOrSmsM, $TitleM, $s_EMailOrSmsR, $TitleR, $DailyReport, $DailyReportFile, $sSPassword, _

$ReportDayDate, $ReportMonthDate, $ReportText, $BeepFlag, $StartPing, $EndPing, $sSTrayExit, $sSTrayMenu, $sSTrayToggle, $sSTrayAlerts, _

$sSTraySound, $StartCycle, $EndCycle, $StartECycle, $EndECycle, $sSTrayStatus, $LogTextS, $LogFile, $OldHour, $ViewFile, _

$ToAddressR, $s_CallFileID, $mprocess, $TextMsg, $ipFlag, $key, $Password, $logtext, $message, $logo, $s_Editor, $s_smtpTransport, _

$mailcheck, $CallsStop, $CallsStart, $s_CallsStop, $s_CallsStart, $CallsFlag, $iRegKey1, $iRegKey2, $iRegKey3, $iRegKey4, $Display, _

$s_SendersAddress, $s_StartCode, $s_StopCode, $ProcessedCalls, $ProcessedMsgs, $ProcessedVoice, $mMailServer, $mGateway, $mRouter, _

$mSkype, $mApplic2, $MServerTxt, $GatewayTxt, $RouterTxt, $SkypeTxt, $Applic2Txt, $StatusMsgFlag

and you exceed a specific number of variables and lines Tidy will fail as it does on the above.

To fix the problem I simply split the Global statement into two to reduce the number of $variables and "_" for multiple lines for each

It now looks like this and it works....

I will leave it up to you as to whether it needs a fix.... Ant..

Hmm, that's interesting. That crashes Tidy for me too.

The total line length for that is only about 2K characters, so it's not the 4K limit I first thought of on seeing that.

Shorter reproducer:

Global $A1, $B1, $C1, $D1, $E1, $F1, $G1, $H1, $I1, $J1, $K1, $L1, $M1, $N1, $O1, $P1, $Q1, $R1, $S1, $T1, $U1, $V1, $W1, $X1, $Y1, $Z1, _
        $A2, $B2, $C2, $D2, $E2, $F2, $G2, $H2, $I2, $J2, $K2, $L2, $M2, $N2, $O2, $P2, $Q2, $R2, $S2, $T2, $U2, $V2, $W2, $X2, $Y2, $Z2, _
        $A3, $B3, $C3, $D3, $E3, $F3, $G3, $H3, $I3, $J3, $K3, $L3, $M3, $N3, $O3, $P3, $Q3, $R3, $S3, $T3, $U3, $V3, $W3, $X3, $Y3, $Z3, _
        $A4, $B4, $C4, $D4, $E4, $F4, $G4, $H4, $I4, $J4, $K4, $L4, $M4, $N4, $O4, $P4, $Q4, $R4, $S4, $T4, $U4, $V4, $W4, $X4, $Y4, $Z4, _
        $A5, $B5, $C5, $D5, $E5, $F5, $G5, $H5, $I5, $J5, $K5, $L5, $M5, $N5, $O5, $P5, $Q5, $R5, $S5, $T5, $U5, $V5, $W5, $X5, $Y5, $Z5, _
        $A6, $B6, $C6, $D6, $E6, $F6, $G6, $H6, $I6, $J6, $K6, $L6, $M6, $N6, $O6, $P6, $Q6, $R6, $S6, $T6, $U6, $V6, $W6, $X6, $Y6, $Z6, _
        $A7, $B7, $C7, $D7, $E7, $F7, $G7, $H7, $I7, $J7, $K7, $L7, $M7, $N7, $O7, $P7, $Q7, $R7, $S7, $T7, $U7, $V7, $W7, $X7, $Y7, $Z7, _
        $A8, $B8, $C8, $D8, $E8, $F8, $G8, $H8, $I8, $J8, $K8, $L8, $M8, $N8, $O8, $P8, $Q8, $R8, $S8, $T8, $U8, $V8, $W8, $X8, $Y8, $Z8, _
        $A9, $B9, $C9, $D9, $E9, $F9, $G9, $H9, $I9, $J9, $K9, $L9, $M9, $N9, $O9, $P9, $Q9, $R9, $S9, $T9, $U9, $V9, $W9, $X9, $Y9, $Z9, _
        $A10, $B10, $C10, $D10, $E10, $F10, $G10, $H10, $I10, $J10, $K10, $L10, $M10, $N10, $O10, $P10, $Q10, $R10, $S10, $T10, $U10, $V10, $W10, $X10, $Y10, $Z10, _
        $A11, $B11, $C11, $D11, $E11, $F11, $G11, $H11, $I11, $J11, $K11, $L11, $M11, $N11, $O11, $P11, $Q11, $R11, $S11, $T11, $U11, $V11, $W11, $X11, $Y11, $Z11, _
        $A12, $B12, $C12, $D12, $E12, $F12, $G12, $H12, $I12, $J12, $K12, $L12, $M12, $N12, $O12, $P12, $Q12, $R12, $S12, $T12, $U12, $V12, $W12, $X12, $Y12, $Z12, _
        $A13, $B13, $C13, $D13, $E13, $F13, $G13, $H13, $I13, $J13, $K13, $L13, $M13, $N13, $O13, $P13, $Q13, $R13, $S13, $T13, $U13, $V13, $W13, $X13, $Y13, $Z13, _
        $A14, $B14, $C14, $D14, $E14, $F14, $G14, $H14, $I14, $J14, $K14, $L14, $M14, $N14, $O14, $P14, $Q14, $R14, $S14, $T14, $U14, $V14, $W14, $X14, $Y14, $Z14, _
        $A15, $B15, $C15, $D15, $E15, $F15, $G15, $H15, $I15, $J15, $K15, $L15, $M15, $N151

The above Global declaration, all by itself, will successfully Tidy in SciTE. But if you add one character to it (change $N151 to $N1512) then it crashes Tidy.

Edited August 19, 2008 by PsaltyDS

[anixon]

Hmm, that's interesting. That crashes Tidy for me too.

The total line length for that is only about 2K characters, so it's not the 4K limit I first thought of on seeing that.

Shorter reproducer:

Global $A1, $B1, $C1, $D1, $E1, $F1, $G1, $H1, $I1, $J1, $K1, $L1, $M1, $N1, $O1, $P1, $Q1, $R1, $S1, $T1, $U1, $V1, $W1, $X1, $Y1, $Z1, _
        $A2, $B2, $C2, $D2, $E2, $F2, $G2, $H2, $I2, $J2, $K2, $L2, $M2, $N2, $O2, $P2, $Q2, $R2, $S2, $T2, $U2, $V2, $W2, $X2, $Y2, $Z2, _
        $A3, $B3, $C3, $D3, $E3, $F3, $G3, $H3, $I3, $J3, $K3, $L3, $M3, $N3, $O3, $P3, $Q3, $R3, $S3, $T3, $U3, $V3, $W3, $X3, $Y3, $Z3, _
        $A4, $B4, $C4, $D4, $E4, $F4, $G4, $H4, $I4, $J4, $K4, $L4, $M4, $N4, $O4, $P4, $Q4, $R4, $S4, $T4, $U4, $V4, $W4, $X4, $Y4, $Z4, _
        $A5, $B5, $C5, $D5, $E5, $F5, $G5, $H5, $I5, $J5, $K5, $L5, $M5, $N5, $O5, $P5, $Q5, $R5, $S5, $T5, $U5, $V5, $W5, $X5, $Y5, $Z5, _
        $A6, $B6, $C6, $D6, $E6, $F6, $G6, $H6, $I6, $J6, $K6, $L6, $M6, $N6, $O6, $P6, $Q6, $R6, $S6, $T6, $U6, $V6, $W6, $X6, $Y6, $Z6, _
        $A7, $B7, $C7, $D7, $E7, $F7, $G7, $H7, $I7, $J7, $K7, $L7, $M7, $N7, $O7, $P7, $Q7, $R7, $S7, $T7, $U7, $V7, $W7, $X7, $Y7, $Z7, _
        $A8, $B8, $C8, $D8, $E8, $F8, $G8, $H8, $I8, $J8, $K8, $L8, $M8, $N8, $O8, $P8, $Q8, $R8, $S8, $T8, $U8, $V8, $W8, $X8, $Y8, $Z8, _
        $A9, $B9, $C9, $D9, $E9, $F9, $G9, $H9, $I9, $J9, $K9, $L9, $M9, $N9, $O9, $P9, $Q9, $R9, $S9, $T9, $U9, $V9, $W9, $X9, $Y9, $Z9, _
        $A10, $B10, $C10, $D10, $E10, $F10, $G10, $H10, $I10, $J10, $K10, $L10, $M10, $N10, $O10, $P10, $Q10, $R10, $S10, $T10, $U10, $V10, $W10, $X10, $Y10, $Z10, _
        $A11, $B11, $C11, $D11, $E11, $F11, $G11, $H11, $I11, $J11, $K11, $L11, $M11, $N11, $O11, $P11, $Q11, $R11, $S11, $T11, $U11, $V11, $W11, $X11, $Y11, $Z11, _
        $A12, $B12, $C12, $D12, $E12, $F12, $G12, $H12, $I12, $J12, $K12, $L12, $M12, $N12, $O12, $P12, $Q12, $R12, $S12, $T12, $U12, $V12, $W12, $X12, $Y12, $Z12, _
        $A13, $B13, $C13, $D13, $E13, $F13, $G13, $H13, $I13, $J13, $K13, $L13, $M13, $N13, $O13, $P13, $Q13, $R13, $S13, $T13, $U13, $V13, $W13, $X13, $Y13, $Z13, _
        $A14, $B14, $C14, $D14, $E14, $F14, $G14, $H14, $I14, $J14, $K14, $L14, $M14, $N14, $O14, $P14, $Q14, $R14, $S14, $T14, $U14, $V14, $W14, $X14, $Y14, $Z14, _
        $A15, $B15, $C15, $D15, $E15, $F15, $G15, $H15, $I15, $J15, $K15, $L15, $M15, $N151

The above Global declaration, all by itself, will successfully Tidy in SciTE. But if you add one character to it (change $N151 to $N1512) then it crashes Tidy.

Please to hear that you were able to confirm that it was not just my system that was the problem.

I guess that it is something that Jos might like to look into and see if its a bug or not.

Perhaps there is a better way of setting the Global variables which does not encourter the problem.

Goods news is that there is a work around so fixing it [if it is absolutely necessary and to stop it being posted again in the future] is probably just a house keeping exercise

Ant..

[Jos]

I will have a look at this to see what is happening... more later.

Jos

[Jos]

Which version of Tidy are you both testing with?

Jos

[PsaltyDS]

Which version of Tidy are you both testing with?

Jos

Aww, good grief!

I had what ever came with the SciTE distribution was dated 2008-05-17. I just downloaded 2008-08-14 today (not related to this), and now I'm not seeing the problem:

Tidy AutoIt3 v2.0.23.21   Copyright (c) Jos van der Zande  August,12 2008

You take all the fun out of it when you fix things before they're reported!

[Jos]

Aww, good grief!

I had what ever came with the SciTE distribution was dated 2008-05-17. I just downloaded 2008-08-14 today (not related to this), and now I'm not seeing the problem:

Tidy AutoIt3 v2.0.23.21   Copyright (c) Jos van der Zande  August,12 2008

You take all the fun out of it when you fix things before they're reported!

It was actually reported but in a different way and the fix also included fixing this bug.

Now why did I at first ASSUME you of all people were testing with the latest before confirming?

Thanks

Edited August 19, 2008 by Jos

[anixon]

It was actually reported but in a different way and the fix also included fixing this bug.

Now why did I at first ASSUME you of all people were testing with the latest before confirming?

Thanks

I am running Tidy 2.0.23.14 and was assuming that it was the latest version given that I downloaded the Scite software last week Ver 1.76 30 Mar 2008 Ant..

[Jos]

I am running Tidy 2.0.23.14 and was assuming that it was the latest version given that I downloaded the Scite software last week Ver 1.76 30 Mar 2008 Ant..

I Released a new SciTE4AutoIt3 installer on 08/10/2008 and 08/14/2008 (last one to fix a bug in Scilexer.dll), but regularly post Beta version of utilities like Tidy when bugs are reported between full installer releases.

Anyways, I was just jokingly challenging PsaltyDS with that remark, nothing serious meant.

Jos

[SmOke_N]

It was actually reported but in a different way and the fix also included fixing this bug.

[mbkowns]

Vista sucks

Del the partition and start over...

[Jos]

Vista sucks

Del the partition and start over...

...and your contribution to this thread is ????

We don't need this kind of comments in these threads.

Jos

[anixon]

...and your contribution to this thread is ????

We don't need this kind of comments in these threads.

Jos

Just to confirm that I downloaded the latest verstion of Tidy and everything is working thanks for your input. Ant..

Edited August 20, 2008 by anixon