AutoIt3Wrapper_GUI: Compile, directives block update.

[anonymous]

• duplicate directive commands.

AutoIt3Wrapper: 2.0.1.24/2.0.1.29. (Environment(Language:0409 Keyboard:00000409 OS:WIN_XP/Service Pack 3 CPU:X86 OS:X86))

Updating the following directives setup,

#Region ;**** Directives created by AutoIt3Wrapper_GUI ****
#AutoIt3Wrapper_AU3Check_Parameters=-d -w 1 -w 2 -w 3 -w 4        -w 6 -q
#AutoIt3Wrapper_Au3Check_Parameters=-d -w 1 -w 2 -w 3 -w 4 -w 5 -w 6 -q
#EndRegion ;**** Directives created by AutoIt3Wrapper_GUI ****

;; note part.
;~ #Region ;**** Directives created by AutoIt3Wrapper_GUI ****
;~ #AutoIt3Wrapper_AU3Check_Parameters=-d -w 1
;~ #EndRegion ;**** Directives created by AutoIt3Wrapper_GUI ****
;~ #AutoIt3Wrapper_Au3Check_Parameters=-d -w 1

result in,

#Region ;**** Directives created by AutoIt3Wrapper_GUI ****
#AutoIt3Wrapper_outfile_type=a3x
#AutoIt3Wrapper_AU3Check_Parameters=-d -w 1 -w 2 -w 3 -w 4 -w 5 -w 6 -q
#EndRegion ;**** Directives created by AutoIt3Wrapper_GUI ****
#AutoIt3Wrapper_Au3Check_Parameters=-d -w 1 -w 2 -w 3 -w 4 -w 5 -w 6 -q

;; note part.
;~ ;~ #AutoIt3Wrapper_AU3Check_Parameters=-d -w 1
;~ ;~ #AutoIt3Wrapper_Au3Check_Parameters=-d -w 1

issue 1a) instead of the first(not used) dup-directive the second(active) dup-directive is used as source for the (assuming)"move not active duplicate out of the way"
issue 1b) the moved line is not commented out.

note 2a) already commented out directives getting additional comment out part.
note 2b) eats inactive AutoIt3Wrapper_GUI region tokes.

[Jos]

I am not really planning to work on issues that are the result of copying statements generated by AutoIt3Wrapper.
Just never copy this line in whatever flavor, shape or form including commented:

#Region ; Directives created by AutoIt3Wrapper_GUI

Jos

[TicketCleanup]

Automatic ticket cleanup.

[anonymous]

Replying to Jos:

I am not really planning to work on issues that are the result of copying statements generated by AutoIt3Wrapper.
Just never copy this line in whatever flavor, shape or form including commented:

#Region ; Directives created by AutoIt3Wrapper_GUI

Jos

I see. ... No I don't.
Why is the first dupe-directive changed?
If its just ignored, and thereby pushed downwards by the creation of a new AutoIt3Wrapper_GUI block. It should stay the same.
Its changed. (and the secondary commented-out AutoIt3Wrapper_GUI block has nothing to do with this)

The secondary commented-out AutoIt3Wrapper_GUI was only added to show that not only active directives are changed, but disabled ones to. although in a none destructive way. That this is a side effect of the AutoIt3Wrapper_GUI block region headers, thats fine by me.

[anonymous]

Looked at the source, but ... well we see.

[anonymous]

Suggestion patch.

3559c3559
< 	$source = StringRegExpReplace($source, '(?s)(?i)' & @CRLF & "(\s?)" & $directive & '(.*?)' & @CRLF, @CRLF)
---
> 	$source = StringRegExpReplace($source, '(?si)(?<=\r\n)\s*' & $directive & '.*?\r\n', @CRLF)

I'm out.

[anonymous]

Bug fix on suggestion patch.

3559c3559
< 	$source = StringRegExpReplace($source, '(?s)(?i)' & @CRLF & "(\s?)" & $directive & '(.*?)' & @CRLF, @CRLF)
---
> 	$source = StringRegExpReplace($source, '(?si)(?<=\r\n)\s*' & $directive & '.*?\r\n', '')

[Jos]

I had a look at why the dup directive was changed and not removed and fixed that.
While in there also change the change for the header and footer #region statements to only remove the non commented lines.

Give it a spin to see if that fixes it.

Jos

[anonymous]

I'm not sure why @CRLF & "(\s?)" .. is used in the regexp's. As \s? will only target/capture one blank character. But \s* will probably has it own set of problems.
Anyway: v2.0.1.31 Works for me.

[anonymous]

you are right...my intent was to ignore leading whitespaces so updated it to (\s*)
Updated the Beta version.

Thanks
jos