Regex] Rounding numbers python script does not run properly

PeterJones

Replace selected text with? :

Maybe Would you like to replace the selected text with the following text?

Alan Kilborn

Well, Pythonscript sizes the dialog box according to contents, so I was trying to keep it as narrow and short as possible to avoid covering too much of the Notepad++ main window. But…I’ve already added more info to make it taller, so why not wider as well? :)

Alan Kilborn

Would you also publish your code for the interactive replace?

I’m still working on a generic version; may never finish it

Follow-up:

There may not be a reasonable way to script this (the “generic version”); here’s why: There doesn’t seem to be a way to do a “boost-ish” replacement option on a Python string. It’s only possible to do it on in-place text.

What I want to do in the code is pull a copy of the matched text into a Python string and then do a Notepad++ -like replacement operation on that. Sure I can do a Python “re” replacement on it, but that isn’t always going to be equivalent.

Digging deeper, I see there is an open issue on this kind of thing in the Pythonscript code pages, here. But it hasn’t been acted on so it isn’t a reality.

If anyone has thoughts on this, I’m all ears. @Ekopalypse ?

Alan Kilborn

Ugh. Clearly I meant this Pythonscript issue: https://github.com/bruderstein/PythonScript/issues/101 in the previous post, but it is too late for an edit. :(

Ekopalypse

@Alan-Kilborn

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.