Hacker News Reader

I'd keep an int flag, initialized to zero.

If you do, your code is buggy: sig_atomic_t is the only type you can safely use inside and outside a signal handler.

Please explain why this would be true for single-threaded applications.

Because your signal handler may get interrupted by another signal.

So what? Even then, there will be no concurrency issues in a single threaded application.