summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--orkaudio/OrkAudio.cpp2
-rw-r--r--orkaudio/audiocaptureplugins/voip/Iax2Session.cpp2
-rw-r--r--orkaudio/audiocaptureplugins/voip/RtpSession.cpp3
-rw-r--r--orkaudio/audiocaptureplugins/voip/SipTcp.cpp8
-rw-r--r--orkaudio/audiocaptureplugins/voip/VoIp.cpp48
-rw-r--r--orkaudio/filters/rtpmixer/RtpMixer.cpp4
-rw-r--r--orkbasecxx/AudioCapture.cpp1
-rw-r--r--orkbasecxx/BatchProcessing.cpp2
-rw-r--r--orkbasecxx/ConfigManager.cpp8
-rw-r--r--orkbasecxx/LogManager.cpp8
-rw-r--r--orkbasecxx/MemUtils.cpp2
-rw-r--r--orkbasecxx/MultiThreadedServer.cpp1
-rw-r--r--orkbasecxx/Utils.cpp2
-rw-r--r--orkbasecxx/audiofile/AudioFile.h2
-rw-r--r--orkbasecxx/audiofile/MediaChunkFile.cpp6
-rw-r--r--orkbasecxx/audiofile/PcmFile.cpp2
-rw-r--r--orkbasecxx/serializers/Serializer.cpp10
-rw-r--r--orkbasecxx/serializers/SingleLineSerializer.cpp6
-rw-r--r--orkbasecxx/serializers/UrlSerializer.cpp6
19 files changed, 56 insertions, 67 deletions
diff --git a/orkaudio/OrkAudio.cpp b/orkaudio/OrkAudio.cpp
index 0866f37..46ec9ac 100644
--- a/orkaudio/OrkAudio.cpp
+++ b/orkaudio/OrkAudio.cpp
@@ -92,7 +92,7 @@ void LoadPlugins(std::list<ACE_DLL>& pluginDlls)
else
{
dirent* dirEntry = NULL;
- while(dirEntry = ACE_OS::readdir(dir))
+ while((dirEntry = ACE_OS::readdir(dir)))
{
CStdString dirEntryFilename = dirEntry->d_name;
int extensionPos = dirEntryFilename.Find(pluginExtension);
diff --git a/orkaudio/audiocaptureplugins/voip/Iax2Session.cpp b/orkaudio/audiocaptureplugins/voip/Iax2Session.cpp
index 6216f71..5df6011 100644
--- a/orkaudio/audiocaptureplugins/voip/Iax2Session.cpp
+++ b/orkaudio/audiocaptureplugins/voip/Iax2Session.cpp
@@ -313,7 +313,7 @@ void Iax2Session::ReportMetadata()
g_captureEventCallBack(event, m_capturePort);
}
-static char *iax2_state_to_str(int iax2state)
+static const char *iax2_state_to_str(int iax2state)
{
switch(iax2state) {
case IAX2_STATE_WAITING:
diff --git a/orkaudio/audiocaptureplugins/voip/RtpSession.cpp b/orkaudio/audiocaptureplugins/voip/RtpSession.cpp
index 654a6e6..4e2303c 100644
--- a/orkaudio/audiocaptureplugins/voip/RtpSession.cpp
+++ b/orkaudio/audiocaptureplugins/voip/RtpSession.cpp
@@ -2032,9 +2032,6 @@ void RtpSessions::MapOtherMediaAddress(RtpSessionRef& session, CStdString& ipAnd
origOrkUid = session->GetOrkUid();
if(session2->OrkUidMatches(origOrkUid))
{
- CStdString logMsg;
- char szEndPointIp[16];
-
if(m_log->isInfoEnabled())
{
CStdString logMsg;
diff --git a/orkaudio/audiocaptureplugins/voip/SipTcp.cpp b/orkaudio/audiocaptureplugins/voip/SipTcp.cpp
index 8b02838..35e2774 100644
--- a/orkaudio/audiocaptureplugins/voip/SipTcp.cpp
+++ b/orkaudio/audiocaptureplugins/voip/SipTcp.cpp
@@ -99,7 +99,7 @@ u_char *SafeBuffer::GetBuffer()
// ============================================================
-static char* memFindAfter(char* toFind, char* start, char* stop)
+static char* memFindAfter(const char* toFind, char* start, char* stop)
{
for(char * ptr = start; (ptr<stop) && (ptr != NULL); ptr = (char *)memchr(ptr+1, toFind[0],(stop - start)))
{
@@ -113,7 +113,7 @@ static char* memFindAfter(char* toFind, char* start, char* stop)
// Implementation of strcasestr() - works like strstr() but
// is case insensitive
-char* memFindStr(char* toFind, char* start, char* stop)
+char* memFindStr(const char* toFind, char* start, char* stop)
{
for(char * ptr = start; (ptr<stop) && (ptr != NULL); ptr = (char *)memchr(ptr+1, toFind[0],(stop - start)))
{
@@ -142,7 +142,7 @@ static char* memFindEOL(char* start, char* limit)
static void memToHex(unsigned char* input, size_t len, CStdString&output)
{
char byteAsHex[10];
- for(int i=0; i<len; i++)
+ for(unsigned int i=0; i<len; i++)
{
sprintf(byteAsHex, "%.2x", input[i]);
output += byteAsHex;
@@ -216,7 +216,7 @@ bool SipTcpStream::SipRequestIsComplete()
char *pBufEnd = pBufStart+m_sipRequest->Size();
char *contentLengthHeader = memFindStr("Content-Length: ", pBufStart, pBufEnd);
char *contentLength = memFindAfter("Content-Length: ", pBufStart, pBufEnd);
- int cLength = 0;
+ unsigned int cLength = 0;
if(!contentLength || !contentLengthHeader)
return false;
diff --git a/orkaudio/audiocaptureplugins/voip/VoIp.cpp b/orkaudio/audiocaptureplugins/voip/VoIp.cpp
index d36b0bc..55a5443 100644
--- a/orkaudio/audiocaptureplugins/voip/VoIp.cpp
+++ b/orkaudio/audiocaptureplugins/voip/VoIp.cpp
@@ -119,7 +119,7 @@ typedef ACE_Singleton<VoIp, ACE_Thread_Mutex> VoIpSingleton;
void memToHex(unsigned char* input, size_t len, CStdString&output)
{
char byteAsHex[10];
- for(int i=0; i<len; i++)
+ for(unsigned int i=0; i<len; i++)
{
sprintf(byteAsHex, "%.2x", input[i]);
output += byteAsHex;
@@ -142,11 +142,11 @@ inline char* memnchr(void *s, int c, size_t len)
return NULL;
}
-static char* memFindStr(char* toFind, char* start, char* stop)
+static char* memFindStr(const char* toFind, char* start, char* stop)
{
for(char * ptr = start; (ptr<stop) && (ptr != NULL); ptr = (char *)memchr(ptr+1, toFind[0],(stop-ptr)))
{
- if(ACE_OS::strncasecmp(toFind, ptr, (strlen(toFind) > (stop-ptr) ? (stop-ptr) : strlen(toFind))) == 0)
+ if(ACE_OS::strncasecmp(toFind, ptr, ((int)strlen(toFind) > (stop-ptr) ? (stop-ptr) : strlen(toFind))) == 0)
{
return (ptr);
}
@@ -155,7 +155,7 @@ static char* memFindStr(char* toFind, char* start, char* stop)
}
// find the address that follows the given search string between start and stop pointers - case insensitive
-char* memFindAfter(char* toFind, char* start, char* stop)
+char* memFindAfter(const char* toFind, char* start, char* stop)
{
for(char * ptr = start; (ptr<stop) && (ptr != NULL); ptr = (char *)memnchr(ptr+1, toFind[0],(stop - ptr)))
{
@@ -639,7 +639,7 @@ bool TryIax2New(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct* ipHeader,
memset(&ies, 0, sizeof(ies));
udp_act_payload_len = (ntohs(udpHeader->len)-sizeof(UdpHeaderStruct));
- if(udp_act_payload_len < sizeof(*fh))
+ if(udp_act_payload_len < (int)sizeof(*fh))
return false; /* Frame too small */
if(!(ntohs(fh->scallno) & 0x8000))
@@ -685,10 +685,10 @@ bool TryIax2New(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct* ipHeader,
* callee (Called Number) */
if(!ies.caller) {
- ies.caller = "WITHELD";
+ ies.caller = (char*)"WITHELD";
} else {
if(!strlen(ies.caller)) {
- ies.caller = "WITHELD";
+ ies.caller = (char*)"WITHELD";
}
}
@@ -721,7 +721,7 @@ bool TryIax2Accept(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct* ipHeade
memset(&ies, 0, sizeof(ies));
udp_act_payload_len = (ntohs(udpHeader->len)-sizeof(UdpHeaderStruct));
- if(udp_act_payload_len < sizeof(*fh))
+ if(udp_act_payload_len < (int)sizeof(*fh))
return false; /* Frame too small */
if(!(ntohs(fh->scallno) & 0x8000))
@@ -771,7 +771,7 @@ bool TryIax2Authreq(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct* ipHead
memset(&ies, 0, sizeof(ies));
udp_act_payload_len = (ntohs(udpHeader->len)-sizeof(UdpHeaderStruct));
- if(udp_act_payload_len < sizeof(*fh))
+ if(udp_act_payload_len < (int)sizeof(*fh))
return false; /* Frame too small */
if(!(ntohs(fh->scallno) & 0x8000))
@@ -838,7 +838,7 @@ bool TryIax2Hangup(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct* ipHeade
memset(&ies, 0, sizeof(ies));
udp_act_payload_len = (ntohs(udpHeader->len)-sizeof(UdpHeaderStruct));
- if(udp_act_payload_len < sizeof(*fh))
+ if(udp_act_payload_len < (int)sizeof(*fh))
return false; /* Frame too small */
if(!(ntohs(fh->scallno) & 0x8000))
@@ -884,7 +884,7 @@ bool TryIax2ControlHangup(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct*
return false;
udp_act_payload_len = (ntohs(udpHeader->len)-sizeof(UdpHeaderStruct));
- if(udp_act_payload_len < sizeof(*fh))
+ if(udp_act_payload_len < (int)sizeof(*fh))
return false; /* Frame too small */
if(!(ntohs(fh->scallno) & 0x8000))
@@ -924,7 +924,7 @@ bool TryIax2Reject(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct* ipHeade
memset(&ies, 0, sizeof(ies));
udp_act_payload_len = (ntohs(udpHeader->len)-sizeof(UdpHeaderStruct));
- if(udp_act_payload_len < sizeof(*fh))
+ if(udp_act_payload_len < (int)sizeof(*fh))
return false; /* Frame too small */
if(!(ntohs(fh->scallno) & 0x8000))
@@ -969,7 +969,7 @@ bool TryIax2FullVoiceFrame(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct*
return false;
udp_act_payload_len = (ntohs(udpHeader->len)-sizeof(UdpHeaderStruct));
- if(udp_act_payload_len < sizeof(*fh))
+ if(udp_act_payload_len < (int)sizeof(*fh))
return false; /* Frame too small */
if(!(ntohs(fh->scallno) & 0x8000))
@@ -1021,8 +1021,7 @@ bool TryIax2MetaTrunkFrame(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct*
struct Iax2MetaTrunkEntry *supermini = NULL;
struct Iax2MetaTrunkEntryTs *mini = NULL;
int content_type = 0; /* 0 means mini frames, 1 means super mini (no timestampes) */
- int frame_ts = 0; /* Timestamp of frame */
- int data_len = 0;
+ unsigned int data_len = 0;
int entries = 0, udp_act_payload_len = 0;
Iax2PacketInfoRef info(new Iax2PacketInfo());
@@ -1030,7 +1029,7 @@ bool TryIax2MetaTrunkFrame(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct*
return false;
udp_act_payload_len = (ntohs(udpHeader->len)-sizeof(UdpHeaderStruct));
- if(udp_act_payload_len < sizeof(*mh))
+ if(udp_act_payload_len < (int)sizeof(*mh))
return false; /* Frame too small */
if(mh->meta != 0)
@@ -1140,7 +1139,7 @@ bool TryIax2MiniVoiceFrame(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct*
return false;
udp_act_payload_len = (ntohs(udpHeader->len)-sizeof(UdpHeaderStruct));
- if(udp_act_payload_len < sizeof(*mini))
+ if(udp_act_payload_len < (int)sizeof(*mini))
return false; /* Frame too small */
if((ntohs(mini->scallno) & 0x8000))
@@ -1210,7 +1209,7 @@ bool TryRtcp(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct* ipHeader, Udp
unsigned short mv = 0;
r = (RtcpCommonHeaderStruct*)((unsigned int *)r + ntohs(r->length) + 1);
- while(r < rtcpEnd && ((rtcpEnd - r) >= sizeof(RtcpCommonHeaderStruct)))
+ while(r < rtcpEnd && ((rtcpEnd - r) >= (int)sizeof(RtcpCommonHeaderStruct)))
{
mv = (r->vpc & 0x00c0) >> 6;
if(mv != 2)
@@ -1241,7 +1240,7 @@ bool TryRtcp(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct* ipHeader, Udp
memset(cname, 0, sizeof(cname));
r = (RtcpCommonHeaderStruct*)((unsigned int *)r + ntohs(r->length) + 1);
- while(r < rtcpEnd && ((rtcpEnd - r) >= sizeof(RtcpCommonHeaderStruct)))
+ while(r < rtcpEnd && ((rtcpEnd - r) >= (int)sizeof(RtcpCommonHeaderStruct)))
{
version = (r->vpc & 0x00c0) >> 6;
p = (r->vpc & 0x0020) >> 5;
@@ -1375,9 +1374,7 @@ bool TryRtp(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct* ipHeader, UdpH
if (rtpHeader->version == 2)
{
- u_short source = ntohs(udpHeader->source);
- u_short dest = ntohs(udpHeader->dest);
- if(!(ntohs(udpHeader->source)%2) && !(ntohs(udpHeader->dest)%2) || DLLCONFIG.m_rtpDetectOnOddPorts) // udp ports usually even
+ if((!(ntohs(udpHeader->source)%2) && !(ntohs(udpHeader->dest)%2)) || DLLCONFIG.m_rtpDetectOnOddPorts) // udp ports usually even
{
if((rtpHeader->pt <= 34 && rtpHeader->pt != 13) || (rtpHeader->pt >= 97 && rtpHeader->pt < 127) )
// pt=13 is CN (Comfort Noise)
@@ -1457,7 +1454,7 @@ bool TrySipBye(EthernetHeaderStruct* ethernetHeader, IpHeaderStruct* ipHeader, U
int sipLength = ntohs(udpHeader->len) - sizeof(UdpHeaderStruct);
char* sipEnd = (char*)udpPayload + sipLength;
- if(sipLength < sizeof(SIP_METHOD_BYE) || sipEnd > (char*)packetEnd)
+ if(sipLength < (int)sizeof(SIP_METHOD_BYE) || sipEnd > (char*)packetEnd)
{
return false;
}
@@ -2786,7 +2783,7 @@ void HandleSkinnyMessage(SkinnyHeaderStruct* skinnyHeader, IpHeaderStruct* ipHea
// Extract Calling and Called number.
char* parties = (char*)(&ccm5CallInfo->parties);
char* partiesPtr = parties;
- long partiesLen = partiesLen = (long)packetEnd - (long)ccm5CallInfo - sizeof(SkCcm5CallInfoStruct);
+ long partiesLen = (long)packetEnd - (long)ccm5CallInfo - sizeof(SkCcm5CallInfoStruct);
CStdString callingParty;
CStdString calledParty;
@@ -3411,7 +3408,7 @@ void VoIp::OpenPcapDirectory(CStdString& path)
else
{
dirent* dirEntry = NULL;
- while(dirEntry = ACE_OS::readdir(dir))
+ while((dirEntry = ACE_OS::readdir(dir)))
{
CStdString dirEntryFilename = dirEntry->d_name;
CStdString pcapExtension = ".pcap";
@@ -3733,7 +3730,6 @@ void VoIp::LoadPartyMaps()
{
FILE *maps = NULL;
char buf[1024];
- int i = 0;
int ln = 0;
CStdString logMsg;
diff --git a/orkaudio/filters/rtpmixer/RtpMixer.cpp b/orkaudio/filters/rtpmixer/RtpMixer.cpp
index c2ce98b..f3e4216 100644
--- a/orkaudio/filters/rtpmixer/RtpMixer.cpp
+++ b/orkaudio/filters/rtpmixer/RtpMixer.cpp
@@ -610,7 +610,7 @@ void RtpMixer::StoreRtpPacket(AudioChunkRef& audioChunk, unsigned int correctedT
unsigned int endRtpTimestamp = correctedTimestamp + audioChunk->GetNumSamples();
if (endRtpTimestamp > m_writeTimestamp)
{
- for(int i=0; i<(endRtpTimestamp - m_writeTimestamp); i++)
+ for(unsigned int i=0; i<(endRtpTimestamp - m_writeTimestamp); i++)
{
*m_writePtr = 0;
@@ -761,7 +761,7 @@ void RtpMixer::HandleMixedOutput(AudioChunkRef &chunk, AudioChunkDetails& detail
chanNo = i+1;
chanIdx = i;
- for(int j = 0; j < (details.m_numBytes/2); j++)
+ for(unsigned int j = 0; j < (details.m_numBytes/2); j++)
{
int sample;
int saveIdx = 0;
diff --git a/orkbasecxx/AudioCapture.cpp b/orkbasecxx/AudioCapture.cpp
index 77b9e95..1cb5ca6 100644
--- a/orkbasecxx/AudioCapture.cpp
+++ b/orkbasecxx/AudioCapture.cpp
@@ -207,7 +207,6 @@ int AudioChunk::GetNumSamples()
double AudioChunk::GetDurationSec()
{
- int i = 0;
return ((double)GetNumSamples())/((double)m_details.m_sampleRate);
}
diff --git a/orkbasecxx/BatchProcessing.cpp b/orkbasecxx/BatchProcessing.cpp
index 560ac3e..81746af 100644
--- a/orkbasecxx/BatchProcessing.cpp
+++ b/orkbasecxx/BatchProcessing.cpp
@@ -320,7 +320,7 @@ void BatchProcessing::ThreadHandler(void *args)
if(CONFIG.m_transcodingSleepEveryNumFrames > 0 && CONFIG.m_transcodingSleepUs > 0)
{
- if(frameSleepCounter >= CONFIG.m_transcodingSleepEveryNumFrames)
+ if(frameSleepCounter >= (unsigned int)CONFIG.m_transcodingSleepEveryNumFrames)
{
frameSleepCounter = 0;
struct timespec ts;
diff --git a/orkbasecxx/ConfigManager.cpp b/orkbasecxx/ConfigManager.cpp
index 85ad41e..1dff7d8 100644
--- a/orkbasecxx/ConfigManager.cpp
+++ b/orkbasecxx/ConfigManager.cpp
@@ -48,7 +48,7 @@ void ConfigManager::Initialize()
try
{
- char* cfgFilename = "";
+ char* cfgFilename = NULL;
char* cfgEnvPath = "";
int cfgAlloc = 0;
@@ -69,19 +69,19 @@ void ConfigManager::Initialize()
}
}
- if(!cfgFilename || !strlen(cfgFilename)) {
+ if(!cfgFilename) {
FILE* file = ACE_OS::fopen(CONFIG_FILE_NAME, "r");
if(file)
{
// config.xml exists in the current directory
- cfgFilename = CONFIG_FILE_NAME;
+ cfgFilename = (char*)CONFIG_FILE_NAME;
fclose(file);
}
else
{
// config.xml could not be found in the current
// directory, try to find it in system configuration directory
- cfgFilename = ETC_CONFIG_FILE_NAME;
+ cfgFilename = (char*)ETC_CONFIG_FILE_NAME;
}
}
diff --git a/orkbasecxx/LogManager.cpp b/orkbasecxx/LogManager.cpp
index 6259a70..f058429 100644
--- a/orkbasecxx/LogManager.cpp
+++ b/orkbasecxx/LogManager.cpp
@@ -30,7 +30,7 @@ void OrkLogManager::Initialize()
{
BasicConfigurator::configure();
- char* logCfgFilename = "";
+ char* logCfgFilename = NULL;
char* cfgEnvPath = "";
int cfgAlloc = 0;
@@ -51,19 +51,19 @@ void OrkLogManager::Initialize()
}
}
- if(!logCfgFilename || !strlen(logCfgFilename)) {
+ if(!logCfgFilename) {
FILE* file = ACE_OS::fopen("logging.properties", "r");
if(file)
{
// logging.properties exists in the current directory
- logCfgFilename = "logging.properties";
+ logCfgFilename = (char*)"logging.properties";
fclose(file);
}
else
{
// logging.properties could not be found in the current
// directory, try to find it in system configuration directory
- logCfgFilename = "/etc/orkaudio/logging.properties";
+ logCfgFilename = (char*)"/etc/orkaudio/logging.properties";
}
}
diff --git a/orkbasecxx/MemUtils.cpp b/orkbasecxx/MemUtils.cpp
index 5fd8994..6b7740e 100644
--- a/orkbasecxx/MemUtils.cpp
+++ b/orkbasecxx/MemUtils.cpp
@@ -22,7 +22,7 @@
void MemToHex(unsigned char* input, size_t len, CStdString&output)
{
char byteAsHex[10];
- for(int i=0; i<len; i++)
+ for(unsigned int i=0; i<len; i++)
{
sprintf(byteAsHex, "%.2x", input[i]);
output += byteAsHex;
diff --git a/orkbasecxx/MultiThreadedServer.cpp b/orkbasecxx/MultiThreadedServer.cpp
index 7be1a1e..485b560 100644
--- a/orkbasecxx/MultiThreadedServer.cpp
+++ b/orkbasecxx/MultiThreadedServer.cpp
@@ -320,7 +320,6 @@ int EventStreamingServer::svc(void)
peer().send(header, header.GetLength());
time_t startTime = time(NULL);
- time_t lastSentTime = time(NULL);
sessionId = EventStreamingSingleton::instance()->GetNewSessionId() + " -";
logMsg.Format("%s Event streaming start", sessionId);
diff --git a/orkbasecxx/Utils.cpp b/orkbasecxx/Utils.cpp
index f06d673..656a3ce 100644
--- a/orkbasecxx/Utils.cpp
+++ b/orkbasecxx/Utils.cpp
@@ -234,7 +234,7 @@ void FileEscapeName(CStdString& in, CStdString& out)
// Translates all the characters that are not in file_ok_chars string into %xx sequences
// %xx specifies the character ascii code in hexadecimal
out = "";
- for (int i = 0 ; i<in.size() ; i++)
+ for (unsigned int i = 0 ; i<in.size() ; i++)
{
if (strchr(file_ok_chars, in.GetAt(i)))
{
diff --git a/orkbasecxx/audiofile/AudioFile.h b/orkbasecxx/audiofile/AudioFile.h
index cc37e4c..ce54385 100644
--- a/orkbasecxx/audiofile/AudioFile.h
+++ b/orkbasecxx/audiofile/AudioFile.h
@@ -41,7 +41,7 @@ public:
/** Writes a chunk of audio to disk.
If stereo capture, this represents the local party */
virtual void WriteChunk(AudioChunkRef chunkRef) = 0;
- /** Writes a chunk of audio from the remote pary to disk (if stereo capture)
+ /** Writes a chunk of audio from the remote pary to disk (if stereo capture) */
//virtual bool WriteRemoteChunk(AudioChunkRef chunkRef) = 0;
/** Reads a chunk of audio stereo-wise
If underlying storage is mono, remoteChunk will be NULL
diff --git a/orkbasecxx/audiofile/MediaChunkFile.cpp b/orkbasecxx/audiofile/MediaChunkFile.cpp
index 3856be5..1e9d6a3 100644
--- a/orkbasecxx/audiofile/MediaChunkFile.cpp
+++ b/orkbasecxx/audiofile/MediaChunkFile.cpp
@@ -61,8 +61,7 @@ bool MediaChunkFile::FlushToDisk()
continue;
}
- int tmp = sizeof(AudioChunkDetails);
- unsigned int numWritten = ACE_OS::fwrite(tmpChunk->GetDetails(), sizeof(AudioChunkDetails), 1, m_stream);
+ int numWritten = ACE_OS::fwrite(tmpChunk->GetDetails(), sizeof(AudioChunkDetails), 1, m_stream);
if(numWritten != 1)
{
writeError = true;
@@ -97,7 +96,7 @@ void MediaChunkFile::WriteChunk(AudioChunkRef chunkRef)
m_chunkQueueDataSize += pChunk->GetNumBytes();
m_chunkQueue.push(chunkRef);
- if(m_chunkQueueDataSize > (CONFIG.m_captureFileBatchSizeKByte*1024))
+ if(m_chunkQueueDataSize > (unsigned int)(CONFIG.m_captureFileBatchSizeKByte*1024))
{
if (m_stream)
{
@@ -117,7 +116,6 @@ void MediaChunkFile::WriteChunk(AudioChunkRef chunkRef)
int MediaChunkFile::ReadChunkMono(AudioChunkRef& chunkRef)
{
unsigned int numRead = 0;
- bool readError = false;
if (m_stream)
{
diff --git a/orkbasecxx/audiofile/PcmFile.cpp b/orkbasecxx/audiofile/PcmFile.cpp
index 85f49e3..c6ba65c 100644
--- a/orkbasecxx/audiofile/PcmFile.cpp
+++ b/orkbasecxx/audiofile/PcmFile.cpp
@@ -59,7 +59,7 @@ void PcmFile::WriteChunk(AudioChunkRef chunkRef)
throw(CStdString("Write attempt on unopened file:")+ m_filename);
}
- if (numWritten != chunkRef->GetNumSamples())
+ if (numWritten != (unsigned int)chunkRef->GetNumSamples())
{
throw(CStdString("Could not write to file:")+ m_filename);
}
diff --git a/orkbasecxx/serializers/Serializer.cpp b/orkbasecxx/serializers/Serializer.cpp
index 3318833..df066b0 100644
--- a/orkbasecxx/serializers/Serializer.cpp
+++ b/orkbasecxx/serializers/Serializer.cpp
@@ -318,7 +318,7 @@ void Serializer::GetCsv(const char* key, std::list<CStdString>& value, bool req
CStdString element;
bool first = true;
GetString(key, stringValue, required);
- for(int i=0; i<stringValue.length(); i++)
+ for(unsigned int i=0; i<stringValue.length(); i++)
{
TCHAR c = stringValue[i];
if(c == ',')
@@ -404,7 +404,7 @@ void Serializer::GetIpRanges(const char* key, IpRanges& value, bool required)
// Escape the comma and percent characters for adding string to csv list
void Serializer::EscapeCsv(CStdString& in, CStdString& out)
{
- for(int i=0; i<in.length();i++)
+ for(unsigned int i=0; i<in.length();i++)
{
TCHAR c = in[i];
if (c == ',')
@@ -425,7 +425,7 @@ void Serializer::EscapeCsv(CStdString& in, CStdString& out)
// Unescape the comma and percent characters when retrieving from csv list
void Serializer::UnEscapeCsv(CStdString& in, CStdString& out)
{
- int iin = 0;
+ unsigned int iin = 0;
while(iin<in.length())
{
@@ -455,7 +455,7 @@ void Serializer::UnEscapeCsv(CStdString& in, CStdString& out)
// Escape the colon and percent characters for adding string to a pair of the form "key:value"
void Serializer::EscapePair(CStdString& in, CStdString& out)
{
- for(int i=0; i<in.length();i++)
+ for(unsigned int i=0; i<in.length();i++)
{
TCHAR c = in[i];
if (c == ':')
@@ -476,7 +476,7 @@ void Serializer::EscapePair(CStdString& in, CStdString& out)
// UnEscape the colon and percent characters after retrieving a key or value from a pair of the form "key:value"
void Serializer::UnEscapePair(CStdString& in, CStdString& out)
{
- int iin = 0;
+ unsigned int iin = 0;
while(iin<in.length())
{
diff --git a/orkbasecxx/serializers/SingleLineSerializer.cpp b/orkbasecxx/serializers/SingleLineSerializer.cpp
index 807f572..f60156d 100644
--- a/orkbasecxx/serializers/SingleLineSerializer.cpp
+++ b/orkbasecxx/serializers/SingleLineSerializer.cpp
@@ -45,7 +45,7 @@ void SingleLineSerializer::DeSerialize(CStdString& input)
input.Trim();
- for(int i=0; i<input.length() && state!= SingleLineErrorState; i++)
+ for(unsigned int i=0; i<input.length() && state!= SingleLineErrorState; i++)
{
TCHAR character = input[i];
@@ -125,7 +125,7 @@ void SingleLineSerializer::DeSerialize(CStdString& input)
// Escape the space, equals and percent characters for serializing to Key-Value-Pair text
void SingleLineSerializer::EscapeSingleLine(CStdString& in, CStdString& out)
{
- for(int i=0; i<in.length();i++)
+ for(unsigned int i=0; i<in.length();i++)
{
TCHAR c = in[i];
if (c == ' ')
@@ -150,7 +150,7 @@ void SingleLineSerializer::EscapeSingleLine(CStdString& in, CStdString& out)
// Unescape the space, equals and percent characters for serializing to Key-Value-Pair text
void SingleLineSerializer::UnEscapeSingleLine(CStdString& in, CStdString& out)
{
- int iin = 0;
+ unsigned int iin = 0;
while(iin<in.length())
{
diff --git a/orkbasecxx/serializers/UrlSerializer.cpp b/orkbasecxx/serializers/UrlSerializer.cpp
index b80024a..6cabccd 100644
--- a/orkbasecxx/serializers/UrlSerializer.cpp
+++ b/orkbasecxx/serializers/UrlSerializer.cpp
@@ -48,7 +48,7 @@ void UrlSerializer::DeSerialize(CStdString& input)
input.Trim();
- for(int i=0; i<input.length() && state!= UrlErrorState; i++)
+ for(unsigned int i=0; i<input.length() && state!= UrlErrorState; i++)
{
TCHAR character = input[i];
@@ -130,7 +130,7 @@ void UrlSerializer::EscapeUrl(CStdString& in, CStdString& out)
// Translates all the characters that are not in url_ok_chars string into %xx sequences
// %xx specifies the character ascii code in hexadecimal
out = "";
- for (int i = 0 ; i<in.size() ; i++)
+ for (unsigned int i = 0 ; i<in.size() ; i++)
{
if (in.GetAt(i) == ' ')
out += '+';
@@ -150,7 +150,7 @@ void UrlSerializer::UnEscapeUrl(CStdString& in, CStdString& out)
// Translates all %xx escaped sequences to corresponding ascii characters
out = "";
char pchHex[3];
- for (int i = 0; i<in.size(); i++)
+ for (unsigned int i = 0; i<in.size(); i++)
{
switch (in.GetAt(i))
{