Trying to add the unit of time in variable name

[mapleFU]

trafficstars

Cron Job is remain unchanged

[torwig]

@mapleFU Can we have int64_t instead of time_t everywhere? Maybe just rename these variables as timestamp and add a comment // in seconds/in milliseconds? WDYT?

[mapleFU]

Both is ok for me. Currently the different "time" is really confusing, any possible change is ok

[mapleFU]

@git-hulk @PragmaTwice Would you mind take a look? I can accept any idea here.

[git-hulk]

@mapleFU It looks great with the unit of time. But some places are using _secs and _sec as the suffix, perhaps we can keep them consistent.

[PragmaTwice]

I think the best idea is to use some types with unit like chrono::duration, but the affected code range will be very large. So currently you can add unit to variable names.

[sonarqubecloud[bot]]

Quality Gate passed

Issues
6 New issues
0 Accepted issues

Measures
0 Security Hotspots
62.9% Coverage on New Code
1.3% Duplication on New Code

See analysis details on SonarCloud