Abstracting some code that is copy pasted a few times.

[NickyDaB]

Summary:

  const int32_t TS_BUFF_SIZE(80);
  const int32_t TS_USEC_SIZE(120);

is referenced a few times in the code.

At least once in these files:

• 706a79e#diff-d88d02e16217487809207c2d3e92de29
• 706a79e#diff-da219aa0af5de5bca52b703114080dcb
• 706a79e#diff-f67f59868971b74c444f3e8c10b02991
• 706a79e#diff-8dcef61fe00e9a1ddf1b4d02966255c4

Instead of defining the const vars in each file. I suggest we make a daemon def include file. Then include that and use those consts.

Advantage:

• If we have to change the value in the future, then we change it in one spot and all values are updated. VS leaving it this way. If we need to update it, then we might accidentally miss one value.

Is your feature request related to a problem? Please describe.

Not really a problem. But the issue is described above in the advantage section.

Describe alternatives you've considered

We could also even make a "construct timestamp function" which could even be more robust.

Issue Source:

• idea spawned in a code review session between @NickyDaB and @besawn

Other Notes:

• This is a good first issue and simple issue that might be included in a 24 or 48 hours sprint.

[NickyDaB]

and related:

67fe0e3

[bogdanscode]

Hi NickyDaB,

I noticed that the issue mentions abstracting the variable 'TS_BUFF_SIZE,' and 'TS_USEC_SIZE.' However, it appears that this variable no longer exists in the current codebase. Could you please clarify if this issue is still relevant, or if there might have been some changes that have rendered this issue obsolete?

Thank you for your guidance!

Best regards,
Bogdan