Macros plugin stops working after 1-2 calls

[MiroslavMatas]

@unzip cudatext-win-x64-1.98.2.0.zip to new folder (e.g. c:\temp\cuda)
start cudatext.exe
install Macros addon + exit
start and import Export of Macro.cuda-macros - one stupid Macro1
exit (just for sure… maybe not neccessary)
start and open attached new.txt
attach Alt+F12 shortcut to Macro1
set position at first row, begin of line (caret top left)
Alt+F12
Alt+F12
Alt+F12 - here we have the problem
new.txt
Export of Macro - cuda-macros.txt
@kvichans

[kvichans]

Yes. Repro problem on my Win.
I see that problem of CT core not of the Macros.

[MiroslavMatas]

Yes. Repro problem on my Win.
I see that problem of CT core not of the Macros.

thanks a lot
will cross my fingers…

[MiroslavMatas]

Yes. Repro problem on my Win.
I see that problem of CT core not of the Macros.

@Alexey-T

[Alexey-T]

@MiroslavMatas
I cannot validate this bugreport,

• seems my PC was hacked (Linux PC, on which I made the Win32 build), Report about regex greedy not working #2520
• I don't have Win10 on this notebook (i am not at home)

if someone can compile Cud on his PC, pls test this bug on new build.

[Alexey-T]

plugin Macros here (linux) cannot set hotkey alt+f12 for macro1 at all.
help to find where is the bug.
how plugin sets hotkey for macro1?
i found this in macros plugin.
help to find small repro.

        # Register new subcommands
        if '|reg|' in acts:
            reg_subs        = 'cuda_macros;run;{}'.format('\n'.join(
                             'Macros: {}\t{}'.format(mcr['nm'],mcr['id']) 
                                 for mcr in self.macros)
                             )
            pass;              #LOG and log('reg_subs={}',reg_subs)
            app.app_proc(app.PROC_SET_SUBCOMMANDS, reg_subs)

@kvichans

[kvichans]

1. Для добавления в Куд всех новых команд (макросов) выполняется код

reg_subs        = 'cuda_macros;run;{}'.format('\n'.join(
                             'Macros: {}\t{}'.format(mcr['nm'],mcr['id']) 
                                 for mcr in self.macros)
                             )
app.app_proc(app.PROC_SET_SUBCOMMANDS, reg_subs)

2. Для привязки одной команды к хоткею работает код

app.dlg_hotkeys('cuda_macros,run,'+str(mcr['id']))

В обоих местах mcr['id'] это число (пятизначное). В первом случае mcr['nm'] это имя макроса.

[Alexey-T]

need to know what API call is bad/broken. need small repro of bug.

[kvichans]

Это поможет только если ошибка ядра в логике работы. Но логика при отработке первых двух вызовов очевидно ТАКАЯ же как при третьем. Здесь подозрение на глобальную ошибку работы с памятью. Вряд ли получиться дать "small repro".

[Alexey-T]

после задания alt+f12 через dlg_hotkeys плаг хоткей видит а куда нет.
надо посмотреть.

[Alexey-T]

@kvichans seems Macros plugin is bad. you write keys.json by hands. dont do it, call dlg_hotkeys only.

your code

        # Clear keys.json
        if '|keys|' in acts and ':' in what:
            # Need delete a key 'cuda_macros,run,NNNNN'
            mcr_id      = what[1+what.index(':'):]
            mcr_key     = 'cuda_macros,run,{}'.format(mcr_id)
            keys_json   = app.app_path(app.APP_DIR_SETTINGS)+os.sep+'keys.json'
            if not os.path.exists(keys_json): return
            keys        = apx._json_loads(open(keys_json).read())
            pass;              #LOG and log('??? key={}',mcr_key)
            if keys.pop(mcr_key, None) is not None:
                pass;          #LOG and log('UPD keys.json deleted key={}',mcr_key)
                open(keys_json, 'w').write(json.dumps(keys, indent=2))

[Alexey-T]

I dont see bug in Cud.

[Alexey-T]

@kvichans I dont want to install Macros from GH, pls update in main addon list

[kvichans]

1. Какое отношение имеет вызов GUI dlg_hotkeys к процессу удаления хоткея плагином. Нельзя править keys.json? ОК, укажите (дайте) API вызов, который это делает.
2. Плагин в ГХ и в СФ (для PlugMan) один и тот же.

[Alexey-T]

вы имеет Вики? да. мне придется как и вам искать в нем. :(
dlg_hotkeys вроде сам пишет в keys.json. проверьте.
смотрите вики.

[Alexey-T]

продолжим тут kvichans/cuda_macros#17