Don't use iterator to loop through workspaces

shezdy / hyprsplit

hyprland plugin for separate sets of workspaces on each monitor

BSD 3-Clause "New" or "Revised" License

74 stars 3 forks source link

Don't use iterator to loop through workspaces #23

Closed zakk4223 closed 1 week ago

zakk4223 commented 1 week ago

moveWorkspaceToMonitor can potentially mutate the workspaces vector, which is bad since this is called inside a loop using an iterator. Convert to a boring for loop with index.

shezdy commented 1 week ago

makes sense. thanks!