-
Notifications
You must be signed in to change notification settings - Fork 15
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
I "ported" this to Linux/Qt #5
Comments
Awesome! Sorry about the code quality, as I'm sure you've found out, this was put together as a quick hack that grew a bit more than expected :). Regarding RTT, I actually measure it based on the time it takes the connect() call to return, which on Windows at least is when the 2nd part of the 3-way handshake has completed (so basically a TCP ping). This way you don't require admin privileges to do ICMP, I think Twitch actually filters ICMP so a regular ping wouldn't be of much use anyway. |
hey mate, im looking at the branch and i see no compile instructions, using |
If you have Qt installed:
the resulting binary should be called TwitchTest |
Hello there, I know this is a old topic but i just compiled it here, when i press start it just does nothing. |
@danieloneill - posting here as you have issues disabled on your repo - testing on Debian, it also does nothing for me after clicking the start button - there does not seem to be an option to select which servers to test. |
The Windows version didn't work on my Linux system even via Wine (just didn't draw the form properly?)
It's not really a port, I just made it look really similar and copypasted a bunch of your code. (At the end of the day it turned out Qt and posix sockets are just too different to recycle most of the meat.)
Quality doesn't work (I don't really know of a Linux equivalent), RTT requires root access, TCP Window size dropdown looks great and does nothing. Besides that, everything works! Unless it doesn't.
https://github.com/danieloneill/TwitchTest
I'm just hoping to get a little mention on the page somewhere for the Linux streamers out there, all 3 of us.
The text was updated successfully, but these errors were encountered: