timez | a cli tool for converting time between timezones | Command Line Interface library

This is problematic:

You want this:

There are a number of errors.

First, you need a global array of struct citizen instead of having a function scoped struct citizen in each function.

In struct citizen, idNr was too small to contain a 10 digit ID number. It has to be [at least] one more to account for the EOS char at the end of the string.

Using fread and fwrite to access the data file. They are for binary files, but you have a text file. You need to use fscanf and fprintf respectively.

Using (e.g.) a [function scoped] char *name;. This is a pointer to a char [array/string] and it is uninitialized. Change this to char name[100];

Using strstr instead of strcmp.

Use of scanf or fscanf was inconsistent. If you had compiled with -Wall -Wextra to enable warnings [gcc], they would have become apparent.

It's bad practice to intermix getch [fgetc] and scanf on the same stream.

For consistency, I've added a prompt function instead of the various printf(...); scanf(...); sequences for prompting the user.

You didn't really use your struct birthday in the code.

Here's the cleaned up code. Some basics have been tested, but there are still some issues I couldn't get to. In most places, I've used #if 0 to denote your old code [vs. my new code]

You don't need to call date twice. Just call it once with TZ set to the timezone you want for that variable.

Replace

First of all I think the whole design is broken. You should consider to reuse an existing countdown widget. (The simplest possible JavaScript countdown timer?)

In your specific case,the issue is your setTimeout. It will run your countdowny method again and again, but always with the global varibales.

So you'd need to refactor and give the element you want to update (and the remaining time) into the countdown function.

Realistically, you can't. When a tab is no longer active, its priority gets lowered, which will impact the performance of JavaScript timers.

The best solution I've seen to this so far is to replace the default timing functions with this: https://github.com/turuslan/HackTimer

Read the docs and then include the scripts in your source.

What this script does is replaces the default timer functions (like setInterval) with Web Workers.

These run independently as a separate process, thus aren't given a lower priority when their parent tab is inactive.

1. Do not have 2 main loops. Neither nested nor threaded. There should be only one loop checking the event queue. If you need threading, then you should create your own event distribution in the main loop or manually manage the event queue. Take and remove all events in main-main loop except those needed in the other one. And the other loop should remove only itsown events.

2. Do not draw directly to display surface. It is better to have one surface that you modify and then you blit it fully or parts of it to the screen and flip()/update() it. Sometimes treating display surface as any other causes problems.

3. Perhaps you should use subsurfaces to simplify your task of filling-in rectangles.

When you tidy your code up a little, the problem should go away or it will be more visible.

Try use this.