1
0
mirror of https://github.com/Zygo/bees.git synced 2025-06-16 17:46:16 +02:00

chatter: use static function to control timestamping behavior

Use a static function instead of embedding side-effects in the constructor
of an unrelated class.

Signed-off-by: Zygo Blaxell <bees@furryterror.org>
(cherry picked from commit 85106bd9a9)
This commit is contained in:
Zygo Blaxell
2017-11-11 14:48:34 -05:00
parent 47805253e6
commit 78d04b1417
2 changed files with 10 additions and 12 deletions

View File

@ -17,7 +17,7 @@ namespace crucible {
static shared_ptr<set<string>> chatter_names;
static const char *SPACETAB = " \t";
static int chatter_prefix_timestamp = 1;
static bool add_prefix_timestamp = true;
static
void
@ -49,11 +49,17 @@ namespace crucible {
{
}
void
Chatter::enable_timestamp(bool prefix_timestamp)
{
add_prefix_timestamp = prefix_timestamp;
}
Chatter::~Chatter()
{
ostringstream header_stream;
if (chatter_prefix_timestamp) {
if (add_prefix_timestamp) {
time_t ltime;
DIE_IF_MINUS_ONE(time(&ltime));
struct tm ltm;
@ -97,11 +103,6 @@ namespace crucible {
c.m_oss.str("");
}
ChatterTimestamp::ChatterTimestamp(int prefix_timestamp)
{
chatter_prefix_timestamp = prefix_timestamp;
}
set<ChatterBox*> ChatterBox::s_boxes;
set<ChatterBox*>& ChatterBox::all_boxes()