If you could retest from master and see if 10 ms is still racy for you, I'm happy to accept a PR that adjusts the time. Or maybe we can make it configurable

If you could retest from master and see if 10 ms is still racy for you, I'm happy to accept a PR that adjusts the time. Or maybe we can make it configurable

That was not the main point of this PR. My focus was on using Qt only, not going to POSIX/plain STL, and having less code for the same functionality.

Ok my mistake. I misunderstood the goal of the PR. I'll give this a test and see if we can merge it in still

@gsohler Would you be willing to test this PR out and see if it solves #37 for you as well?
I think I prefer the Qt approach for fstl in this case

#37 is closed already , suppose the answer was yes
…
On Fri, May 26, 2023 at 6:34 PM Paul T @.> wrote: @gsohler https://github.com/gsohler Would you be willing to test this PR out and see if it solves #37 <#37> for you as well? I think I prefer the Qt approach for fstl in this case — Reply to this email directly, view it on GitHub <#107 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCO4MV6J7BVJJDZCQBHP6TXIDLRPANCNFSM6AAAAAAYP7GDGQ . You are receiving this because you were mentioned.Message ID: @.>

I closed that due to your PR. This PR addresses the same issue but in a different way and does not include the changes you made. In retrospect I think I prefer the Qt-centric approach to solve the issue.

i am fine with that. i am happy, that my proposed algorithm is still inside.
…
On Fri, May 26, 2023 at 9:07 PM Paul T @.> wrote: #37 <#37> is closed already , suppose the answer was yes … <#m_8963716526631296567_> On Fri, May 26, 2023 at 6:34 PM Paul T @.> wrote: @gsohler https://github.com/gsohler https://github.com/gsohler https://github.com/gsohler Would you be willing to test this PR out and see if it solves #37 <#37> <#37 <#37>> for you as well? I think I prefer the Qt approach for fstl in this case — Reply to this email directly, view it on GitHub <#107 (comment) <#107 (comment)>>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCO4MV6J7BVJJDZCQBHP6TXIDLRPANCNFSM6AAAAAAYP7GDGQ https://github.com/notifications/unsubscribe-auth/ACCO4MV6J7BVJJDZCQBHP6TXIDLRPANCNFSM6AAAAAAYP7GDGQ . You are receiving this because you were mentioned.Message ID: @.> I closed that due to your PR. This PR addresses the same issue but in a different way and does not include the changes you made. In retrospect I think I prefer the Qt-centric approach to solve the issue. — Reply to this email directly, view it on GitHub <#107 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ACCO4MVI6T4CFENQRN4S3OTXID5PXANCNFSM6AAAAAAYP7GDGQ . You are receiving this because you were mentioned.Message ID: @.**>

Thanks for being flexible