Skip to content
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

For GUI For GoodbyeDPI #127

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

Include-sys
Copy link

Hello,

As you know I'm developing GUI for GoodbyeDPI. In order to properly work(actually just getting stdout texts) stdout functions(printf,puts etc.) have to be flushed after prints. I added fflush() function after every stdout function.

@ValdikSS
Copy link
Owner

That's not a proper way to do that. Have you tried to disable buffering on read in your code?

@ValdikSS
Copy link
Owner

@Include-sys
Copy link
Author

Include-sys commented Feb 24, 2019

I'm not working with winapi, just using Qts features. So you need to flush your stdout or I have to recompile it every time. :) This pull request can be closed If you decide not to do.

@Viktor45
Copy link
Contributor

Viktor45 commented Aug 25, 2024

I'm not working with winapi, just using Qts features. So you need to flush your stdout or I have to recompile it every time. :) This pull request can be closed If you decide not to do.

Why haven't you published the latest source code of your program? I suggest you continue by publishing it, it could help others in analyzing what you want and why in this PR)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants