// wxLog class implementation
// ----------------------------------------------------------------------------
-/* static */
-unsigned wxLog::DoLogNumberOfRepeats()
+unsigned wxLog::LogLastRepetitionCountIfNeeded()
{
- long retval = ms_prevCounter;
- wxLog *pLogger = GetActiveTarget();
- if ( pLogger && ms_prevCounter > 0 )
+ wxCRIT_SECT_LOCKER(lock, ms_prevCS);
+
+ const unsigned count = ms_prevCounter;
+
+ if ( ms_prevCounter )
{
wxString msg;
#if wxUSE_INTL
ms_prevCounter),
ms_prevCounter);
#else
- msg.Printf(wxT("The previous message was repeated."));
+ msg.Printf(wxT("The previous message was repeated %lu times."),
+ ms_prevCounter);
#endif
ms_prevCounter = 0;
ms_prevString.clear();
- pLogger->DoLog(ms_prevLevel, msg, ms_prevTimeStamp);
+ DoLog(ms_prevLevel, msg, ms_prevTimeStamp);
}
- return retval;
+
+ return count;
}
wxLog::~wxLog()
{
- if ( ms_prevCounter > 0 )
- {
- // looks like the repeat count has not been logged yet,
- // so let's do it now
- wxLog::DoLogNumberOfRepeats();
- }
+ LogLastRepetitionCountIfNeeded();
}
/* static */
wxLog *pLogger = GetActiveTarget();
if ( pLogger )
{
- if ( GetRepetitionCounting() && ms_prevString == szString )
+ if ( GetRepetitionCounting() )
{
- ms_prevCounter++;
- }
- else
- {
- if ( GetRepetitionCounting() )
+ wxCRIT_SECT_LOCKER(lock, ms_prevCS);
+
+ if ( szString == ms_prevString )
{
- DoLogNumberOfRepeats();
+ ms_prevCounter++;
+
+ // nothing else to do, in particular, don't log the
+ // repeated message
+ return;
}
+
+ pLogger->LogLastRepetitionCountIfNeeded();
+
+ // reset repetition counter for a new message
ms_prevString = szString;
ms_prevLevel = level;
ms_prevTimeStamp = t;
- pLogger->DoLog(level, szString, t);
}
+
+ pLogger->DoLog(level, szString, t);
}
}
}
// static variables
// ----------------------------------------------------------------------------
+#if wxUSE_THREADS
+wxCriticalSection wxLog::ms_prevCS;
+#endif // wxUSE_THREADS
bool wxLog::ms_bRepetCounting = false;
wxString wxLog::ms_prevString;
unsigned int wxLog::ms_prevCounter = 0;