[ENH] Add character limit warnings to blocks to prevent overflowing an action
BenJ180 opened this issue ยท 6 comments
The latest update does nothing :( Can't even open config with /gse.
Bugsuck output:
22x FrameXML\RestrictedExecution.lua:431: Call failed: FrameXML\RestrictedExecution.lua:126: [string "name, macros = self:GetName(), newtable([=======[/run sfx=GetCV..."]:6490: function or expression too complex near '[=======[/run sfx=GetCVar("Sound_EnableSFX");
/run ers=GetCVar("Sound_EnableErrorSpeech");
/console Sound_EnableSFX 0
/console Sound_EnableErrorSpeech 0
/stopmacro [@playertarget, noexists]
/run sfx=GetCVar("Sound_EnableSFX");
/run ers=GetCVar("Sound_EnableErrorSpeech");
/console Sound_EnableSFX 0
/console Sound_EnableErrorSpeech 0
/targetenemy [noharm][dead]
/cast [nomod, nocombat,stance:1] Charge
/castsequence [nomod, stance:1,combat] reset=combat Devastate, Thunder Clap, Defensive Stance, Null
/castsequence [nomod, stance:1,combat] reset=combat Bloodrage, Null
/cast [modifier:alt,stance:3] Intercept
/cast [modifier:alt,nostance:3] Berserker Stance
/cast [modifier:alt,stance:3] Bloodrage
/cast [nomod, stance:3, combat] Defensive Stance
/cast [nomod, stance:2, combat] Shield Slam
/cast [nomod, nostance:1, nocombat] Battle Stance
/startattack
/run SetCVar("Sound_EnableSFX",sfx);
/run SetCVar("Sound_EnableErrorSpeech",ers);
/script UIErrorsFrame:Hide();
/run UIErrorsFrame:Clear()
/run SetCVar("Sound_EnableSFX",sfx);
/run SetCVar("Sound_EnableErrorSpeech",ers);
/run UIErrorsFrame:Clear()]=======]'
[string "=[C]"]: ?
[string "@FrameXML\RestrictedExecution.lua"]:431: in function <FrameXML\RestrictedExecution.lua:420>
[string "=(tail call)"]: ?
[string "=(tail call)"]: ?
[string "@FrameXML\SecureHandlers.lua"]:499: in function <FrameXML\SecureHandlers.lua:473>
[string "=[C]"]: in function SetAttribute' [string "@FrameXML\SecureHandlers.lua"]:743: in function <FrameXML\SecureHandlers.lua:725> [string "=(tail call)"]: ? [string "@GSE\API\Storage.lua"]:1479: in function CreateGSE3Button'
[string "@gse\API\Storage.lua"]:513: in function OOCUpdateSequence' [string "@GSE\API\Events.lua"]:459: in function ?'
[string "@ElvUI\Libraries\Ace3\AceTimer-3.0\AceTimer-3.0-17.lua"]:55: in function <...s\ElvUI\Libraries\Ace3\AceTimer-3.0\AceTimer-3.0.lua:50>
Locals:
(*temporary) = "Call failed: FrameXML\RestrictedExecution.lua:126: [string "name, macros = self:GetName(), newtable([=======[/run sfx=GetCV..."]:6490: function or expression too complex near '[=======[/run sfx=GetCVar("Sound_EnableSFX");
/run ers=GetCVar("Sound_EnableErrorSpeech");
/console Sound_EnableSFX 0
/console Sound_EnableErrorSpeech 0
/stopmacro [@playertarget, noexists]
/run sfx=GetCVar("Sound_EnableSFX");
/run ers=GetCVar("Sound_EnableErrorSpeech");
/console Sound_EnableSFX 0
/console Sound_EnableErrorSpeech 0
/targetenemy [noharm][dead]
/cast [nomod, nocombat,stance:1] Charge
/castsequence [nomod, stance:1,combat] reset=combat Devastate, Thunder Clap, Defensive Stance, Null
/castsequence [nomod, stance:1,combat] reset=combat Bloodrage, Null
/cast [modifier:alt,stance:3] Intercept
/cast [modifier:alt,nostance:3] Berserker Stance
/cast [modifier:alt,stance:3] Bloodrage
/cast [nomod, stance:3, combat] Defensive Stance
/cast [nomod, stance:2, combat] Shield Slam
/cast [nomod, nostance:
Your macro is too long per action. I would suggest going to GSE's Options and turning off the things like Sound Error Prevention and Then I would look at your macro macro itself. That wont work as you have everything on one action.
You might want to look at https://github.com/TimothyLuke/GnomeSequencer-Enhanced/wiki/Understanding-GSE3-from-a-GSE2-Users-Perspective
Ok thanks Timothy. But that wouldn't prevent me accessing options via /gse would it?
Found the limit here: https://stackoverflow.com/questions/54609218/solution-for-function-or-expression-too-complex-in-string-char
A Macro can only have 255 Actions when compiled.
The warning is in https://github.com/TimothyLuke/GnomeSequencer-Enhanced/releases/tag/3.0.10 The question then is should the macro be truncated at 255 actions or should it fail as it does now.