Regex] Rounding numbers python script does not run properly

Ekopalypse

one idea might be to search for matches and replace only those which are
in the same range as the selected text. Something like

if start_pos_selected text <= match_start and end_pos_selected_text >= match_end:
    replaceSelText

Or did I misunderstood your goal? :-(

Alan Kilborn

@Ekopalypse said:

did I misunderstood your goal?

Maybe that part. :)

To be able to offer a preview of the yet-to-be-done replacement, I need to know what that replacement will be. I can’t do that without actually doing the replacement. Catch-22. I could do it and then undo it, I suppose, but that gets messy. Mostly it should be OK, but sometimes with regexes Boost can differ from Python’s re.

Ekopalypse

@Alan-Kilborn

what about using a hidden scintilla instance?

Alan Kilborn

@Ekopalypse said:

what about using a hidden scintilla instance?

That also sounds “messy” (or “involved”) but if you have some snippet to show me it is not so bad…?

Ekopalypse

@Alan-Kilborn

I used it here.

Ekopalypse

@Alan-Kilborn

just be careful about notepad.destroyScintilla(self.hidden_scintilla)
Currently npp crashes when using this and then request another buffer.
There is already a merged fix but only available on next release.

Alan Kilborn

@Ekopalypse

This is the issue, I assume: https://github.com/notepad-plus-plus/notepad-plus-plus/issues/4487

Ekopalypse

@Alan-Kilborn

correct, that is the one.

Alan Kilborn

@Ekopalypse said:

what about using a hidden scintilla instance?

So, like this: hidden = notepad.createScintilla()

That is actually a nice solution. The hidden instance’s entire text can be set to the matched text from the editor.research(). Then the hidden.rereplace() can be called and the result of the replacement read out into a Python string variable. Thanks, Eko; clearly I would not have thought of this technique without your pointer.

just be careful about notepad.destroyScintilla()

This is not a problem as I will do this near the top of the script, and never do a “destroy”:

try:
    hidden
except NameError:
    hidden = notepad.createScintilla()

and never do a “destroy”

Alan Kilborn

@Alan-Kilborn said:

I will do this near the top of the script, and never do a “destroy”:

and never do a “destroy”

There really is an “echo” in here, or just me repeating myself and not noticing.

Ekopalypse

@Alan-Kilborn

and never do a “destroy”

:-D never ever - don’t even THINK about it unless your npp version is > 7.7.1 :-D

The hidden instance’s entire text can be set to the matched text from the editor.research(). Then the hidden.rereplace() can b

I guess, hidden.rereplace() is enough if you keep the text from editor and hidden in sync.
The replace function gets a match object which contains the position of the text to be replaced so from the replace function you could start the dialog, or?

Alan Kilborn

@Ekopalypse

No, I’m not trying to “parallel” the text in the real editor and the hidden. If I set as the hidden editor’s entire text the match from the search, doing the replace on that guarantees the replace will succeed and the entire text left in the hidden editor’s buffer will be the replacement preview text. Long to explain but it seems to work.