Skip to content

Fixing edge-case bug for history#169

Open
Windows81 wants to merge 2 commits intoF3XTeam:developmentfrom
Windows81:patch-2
Open

Fixing edge-case bug for history#169
Windows81 wants to merge 2 commits intoF3XTeam:developmentfrom
Windows81:patch-2

Conversation

@Windows81
Copy link

@Windows81 Windows81 commented Jul 15, 2024

I occasionally get this error. It happens sporadically whenever I undo and I'm working on a fix for it. I think that it's a race-condition issue.

When it happens, History.Index is greater than #History.Stack. I believe that all the entries in History.Stack were valid when they were added in History.Add.

Players.VisualPlugin.Backpack.Building Tools.Core.History:31: attempt to index nil with 'Unapply'
    Stack Begin
    Script 'Players.VisualPlugin.Backpack.Building Tools.Core.History', Line 31 - function Undo
    Script 'Players.VisualPlugin.Backpack.Building Tools.Core', Line 144
    Script 'Players.VisualPlugin.Backpack.Building Tools.Libraries.SupportLibrary', Line 522
    Stack End

@Windows81 Windows81 changed the title Fixing edge-case bug for history. Fixing edge-case bug for history Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant