I would change
Code:
void WINAPI DLLExport GetDebugItem(LPSTR pBuffer, LPRDATA rdPtr, int id)
{
#if !defined(RUN_ONLY)
char temp[DB_BUFFERSIZE];
switch (id)
{
case DB_NUMMOVINGOBJECTS:
LoadString(hInstLib, IDS_NUMMOVING, temp, DB_BUFFERSIZE);
wsprintf(pBuffer, temp, rdPtr->Objects.size());
break;
}
#endif // !defined(RUN_ONLY)
}
to
Code:
void WINAPI DLLExport GetDebugItem(LPSTR pBuffer, LPRDATA rdPtr, int id)
{
#if !defined(RUN_ONLY)
switch (id)
{
case DB_NUMMOVINGOBJECTS:
wsprintf(pBuffer, "Moving objects: %d", rdPtr->Objects.size());
break;
}
#endif // !defined(RUN_ONLY)
}
as I see no point in using a resource string for that.