1
0
mirror of https://github.com/f4exb/sdrangel.git synced 2025-09-05 06:37:51 -04:00

FileInput: some code cosmetic changes

This commit is contained in:
f4exb 2020-07-11 06:56:32 +02:00
parent c75c35acad
commit 7cddf7ce0b
2 changed files with 30 additions and 19 deletions

View File

@ -122,7 +122,8 @@ void FileInput::openFileStream()
m_recordLength = 0; m_recordLength = 0;
} }
if (getMessageQueueToGUI()) { if (getMessageQueueToGUI())
{
MsgReportHeaderCRC *report = MsgReportHeaderCRC::create(crcOK); MsgReportHeaderCRC *report = MsgReportHeaderCRC::create(crcOK);
getMessageQueueToGUI()->push(report); getMessageQueueToGUI()->push(report);
} }
@ -139,7 +140,8 @@ void FileInput::openFileStream()
<< " center frequency: " << m_centerFrequency << " Hz" << " center frequency: " << m_centerFrequency << " Hz"
<< " sample size: " << m_sampleSize << " bits"; << " sample size: " << m_sampleSize << " bits";
if (getMessageQueueToGUI()) { if (getMessageQueueToGUI())
{
MsgReportFileInputStreamData *report = MsgReportFileInputStreamData::create(m_sampleRate, MsgReportFileInputStreamData *report = MsgReportFileInputStreamData::create(m_sampleRate,
m_sampleSize, m_sampleSize,
m_centerFrequency, m_centerFrequency,
@ -184,12 +186,14 @@ bool FileInput::start()
QMutexLocker mutexLocker(&m_mutex); QMutexLocker mutexLocker(&m_mutex);
qDebug() << "FileInput::start"; qDebug() << "FileInput::start";
if (m_ifstream.tellg() != (std::streampos)0) { if (m_ifstream.tellg() != (std::streampos)0)
{
m_ifstream.clear(); m_ifstream.clear();
m_ifstream.seekg(sizeof(FileRecord::Header), std::ios::beg); m_ifstream.seekg(sizeof(FileRecord::Header), std::ios::beg);
} }
if(!m_sampleFifo.setSize(m_settings.m_accelerationFactor * m_sampleRate * sizeof(Sample))) { if (!m_sampleFifo.setSize(m_settings.m_accelerationFactor * m_sampleRate * sizeof(Sample)))
{
qCritical("Could not allocate SampleFifo"); qCritical("Could not allocate SampleFifo");
return false; return false;
} }
@ -204,7 +208,8 @@ bool FileInput::start()
mutexLocker.unlock(); mutexLocker.unlock();
qDebug("FileInput::startInput: started"); qDebug("FileInput::startInput: started");
if (getMessageQueueToGUI()) { if (getMessageQueueToGUI())
{
MsgReportFileSourceAcquisition *report = MsgReportFileSourceAcquisition::create(true); // acquisition on MsgReportFileSourceAcquisition *report = MsgReportFileSourceAcquisition::create(true); // acquisition on
getMessageQueueToGUI()->push(report); getMessageQueueToGUI()->push(report);
} }
@ -226,7 +231,8 @@ void FileInput::stop()
m_deviceDescription.clear(); m_deviceDescription.clear();
if (getMessageQueueToGUI()) { if (getMessageQueueToGUI())
{
MsgReportFileSourceAcquisition *report = MsgReportFileSourceAcquisition::create(false); // acquisition off MsgReportFileSourceAcquisition *report = MsgReportFileSourceAcquisition::create(false); // acquisition off
getMessageQueueToGUI()->push(report); getMessageQueueToGUI()->push(report);
} }
@ -329,7 +335,7 @@ bool FileInput::handleMessage(const Message& message)
MsgConfigureFileInputWork& conf = (MsgConfigureFileInputWork&) message; MsgConfigureFileInputWork& conf = (MsgConfigureFileInputWork&) message;
bool working = conf.isWorking(); bool working = conf.isWorking();
if (m_fileInputWorker != 0) if (m_fileInputWorker)
{ {
if (working) { if (working) {
startWorker(); startWorker();
@ -352,7 +358,7 @@ bool FileInput::handleMessage(const Message& message)
{ {
MsgReportFileInputStreamTiming *report; MsgReportFileInputStreamTiming *report;
if (m_fileInputWorker != 0) if (m_fileInputWorker)
{ {
if (getMessageQueueToGUI()) if (getMessageQueueToGUI())
{ {
@ -370,8 +376,7 @@ bool FileInput::handleMessage(const Message& message)
if (cmd.getStartStop()) if (cmd.getStartStop())
{ {
if (m_deviceAPI->initDeviceEngine()) if (m_deviceAPI->initDeviceEngine()) {
{
m_deviceAPI->startDeviceEngine(); m_deviceAPI->startDeviceEngine();
} }
} }

View File

@ -35,8 +35,8 @@ FileInputWorker::FileInputWorker(std::ifstream *samplesStream,
QObject(parent), QObject(parent),
m_running(false), m_running(false),
m_ifstream(samplesStream), m_ifstream(samplesStream),
m_fileBuf(0), m_fileBuf(nullptr),
m_convertBuf(0), m_convertBuf(nullptr),
m_bufsize(0), m_bufsize(0),
m_chunksize(0), m_chunksize(0),
m_sampleFifo(sampleFifo), m_sampleFifo(sampleFifo),
@ -49,7 +49,7 @@ FileInputWorker::FileInputWorker(std::ifstream *samplesStream,
m_throttlems(FILESOURCE_THROTTLE_MS), m_throttlems(FILESOURCE_THROTTLE_MS),
m_throttleToggle(false) m_throttleToggle(false)
{ {
assert(m_ifstream != 0); assert(m_ifstream != nullptr);
} }
FileInputWorker::~FileInputWorker() FileInputWorker::~FileInputWorker()
@ -58,11 +58,11 @@ FileInputWorker::~FileInputWorker()
stopWork(); stopWork();
} }
if (m_fileBuf != 0) { if (m_fileBuf) {
free(m_fileBuf); free(m_fileBuf);
} }
if (m_convertBuf != 0) { if (m_convertBuf) {
free(m_convertBuf); free(m_convertBuf);
} }
} }
@ -123,7 +123,7 @@ void FileInputWorker::setBuffers(std::size_t chunksize)
m_bufsize = chunksize; m_bufsize = chunksize;
int nbSamples = m_bufsize/(2 * m_samplebytes); int nbSamples = m_bufsize/(2 * m_samplebytes);
if (m_fileBuf == 0) if (!m_fileBuf)
{ {
qDebug() << "FileInputThread::setBuffers: Allocate file buffer"; qDebug() << "FileInputThread::setBuffers: Allocate file buffer";
m_fileBuf = (quint8*) malloc(m_bufsize); m_fileBuf = (quint8*) malloc(m_bufsize);
@ -133,10 +133,13 @@ void FileInputWorker::setBuffers(std::size_t chunksize)
qDebug() << "FileInputThread::setBuffers: Re-allocate file buffer"; qDebug() << "FileInputThread::setBuffers: Re-allocate file buffer";
quint8 *buf = m_fileBuf; quint8 *buf = m_fileBuf;
m_fileBuf = (quint8*) realloc((void*) m_fileBuf, m_bufsize); m_fileBuf = (quint8*) realloc((void*) m_fileBuf, m_bufsize);
if (!m_fileBuf) free(buf);
if (!m_fileBuf) {
free(buf);
}
} }
if (m_convertBuf == 0) if (!m_convertBuf)
{ {
qDebug() << "FileInputThread::setBuffers: Allocate conversion buffer"; qDebug() << "FileInputThread::setBuffers: Allocate conversion buffer";
m_convertBuf = (quint8*) malloc(nbSamples*sizeof(Sample)); m_convertBuf = (quint8*) malloc(nbSamples*sizeof(Sample));
@ -146,7 +149,10 @@ void FileInputWorker::setBuffers(std::size_t chunksize)
qDebug() << "FileInputThread::setBuffers: Re-allocate conversion buffer"; qDebug() << "FileInputThread::setBuffers: Re-allocate conversion buffer";
quint8 *buf = m_convertBuf; quint8 *buf = m_convertBuf;
m_convertBuf = (quint8*) realloc((void*) m_convertBuf, nbSamples*sizeof(Sample)); m_convertBuf = (quint8*) realloc((void*) m_convertBuf, nbSamples*sizeof(Sample));
if (!m_convertBuf) free(buf);
if (!m_convertBuf) {
free(buf);
}
} }
qDebug() << "FileInputThread::setBuffers: size: " << m_bufsize qDebug() << "FileInputThread::setBuffers: size: " << m_bufsize