Linux: Use list instead of vector for file descriptor callback storage in internal message queue to fix issues when recursively adding or removing callbacks

modules/juce_core/system/juce_StandardHeader.h

@@ -59,6 +59,7 @@
 #include <mutex>
 #include <condition_variable>
 #include <queue>
+#include <list>
 
 //==============================================================================
 #include "juce_CompilerSupport.h"

modules/juce_events/native/juce_linux_Messaging.cpp

@@ -112,10 +112,7 @@ JUCE_IMPLEMENT_SINGLETON (InternalMessageQueue)
 struct InternalRunLoop
 {
 public:
-    InternalRunLoop()
-    {
-        fdReadCallbacks.reserve (8);
-    }
+    InternalRunLoop() = default;
 
     void registerFdCallback (int fd, std::function<void(int)>&& cb, short eventMask)
     {
@@ -186,7 +183,7 @@ public:
 private:
     CriticalSection lock;
 
-    std::vector<std::pair<int, std::function<void(int)>>> fdReadCallbacks;
+    std::list<std::pair<int, std::function<void(int)>>> fdReadCallbacks;
     std::vector<pollfd> pfds;
 };