diff options
author | Tor Lillqvist <tml@collabora.com> | 2015-07-02 16:26:09 +0300 |
---|---|---|
committer | Tor Lillqvist <tml@collabora.com> | 2015-07-02 16:26:47 +0300 |
commit | aec0c5fdaa2d2e51c2f01a3b12d2c77375ec5898 (patch) | |
tree | 92349defd8396a8a20139d80cc88c92933fa54ca | |
parent | Bump version after tarball (diff) | |
download | online-aec0c5fdaa2d2e51c2f01a3b12d2c77375ec5898.tar.gz online-aec0c5fdaa2d2e51c2f01a3b12d2c77375ec5898.zip |
Add a FIXME, found by looking, no time to investigate
distro/collabora/milestone-2
-rw-r--r-- | loolwsd/LOOLSession.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/loolwsd/LOOLSession.cpp b/loolwsd/LOOLSession.cpp index d151a7f3de..30af0e3aec 100644 --- a/loolwsd/LOOLSession.cpp +++ b/loolwsd/LOOLSession.cpp @@ -607,6 +607,7 @@ void MasterProcessSession::dispatchChild() sendTextFrame("error: cmd=load kind=internal"); // We did not use the child session after all + // FIXME: Why do we do the same thing both here and then when we catch the IOException that we throw, a dozen line below? lock.lock(); _availableChildSessions.insert(childSession); std::cout << Util::logPrefix() << "_availableChildSessions size=" << _availableChildSessions.size() << std::endl; @@ -623,6 +624,7 @@ void MasterProcessSession::dispatchChild() Application::instance().logger().error(Util::logPrefix() + "Copying failed: " + exc.message()); sendTextFrame("error: cmd=load kind=failed"); + // FIXME: See above FIXME lock.lock(); _availableChildSessions.insert(childSession); std::cout << Util::logPrefix() << "_availableChildSessions size=" << _availableChildSessions.size() << std::endl; |