GGLinnk/bees
1
0
Fork 0
mirror of https://github.com/Zygo/bees.git synced 2025-05-18 05:45:45 +02:00
Code Packages

chatter: use static function to control timestamping behavior

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

Signed-off-by: Zygo Blaxell <bees@furryterror.org>
This commit is contained in:
Zygo Blaxell 2017-11-11 14:48:34 -05:00
parent bb273770c5
commit 85106bd9a9
2 changed files with 10 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
include/crucible/chatter.h
View File

@ -45,6 +45,8 @@ namespace crucible {
template <class T> Chatter &operator<<(const T& arg);
~Chatter();
static void enable_timestamp(bool prefix_timestamp);
};
template <class Argument>
@ -86,11 +88,6 @@ namespace crucible {
}
};
class ChatterTimestamp {
public:
ChatterTimestamp(int);
};
class ChatterBox {
string m_file;
int m_line;

15
lib/chatter.cc
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()