Additional work to clean up warnings and make the code more robust.

This commit is contained in:
Collin Kidder
2015-08-26 22:22:24 -04:00
parent 004b430ca7
commit 629ad5ec6c
4 changed files with 40 additions and 24 deletions
+1 -1
View File
@@ -15,7 +15,7 @@ void DBCHandler::loadDBCFile(QString filename)
QString line;
QRegularExpression regex;
QRegularExpressionMatch match;
DBC_MESSAGE *currentMessage;
DBC_MESSAGE *currentMessage = NULL;
qDebug() << "DBC File: " << filename;
+2 -2
View File
@@ -88,8 +88,8 @@ void FileComparatorWindow::calculateDetails()
{
QMap<int, FrameData> interestedIDs;
QMap<int, FrameData> referenceIDs;
QTreeWidgetItem *interestedOnlyBase, *referenceOnlyBase, *sharedBase, *bitmapBaseInterested, *bitmapBaseReference;
QTreeWidgetItem *valuesBase, *detail, *sharedItem, *valuesInterested, *valuesReference;
QTreeWidgetItem *interestedOnlyBase, *referenceOnlyBase = NULL, *sharedBase, *bitmapBaseInterested, *bitmapBaseReference = NULL;
QTreeWidgetItem *valuesBase, *detail, *sharedItem, *valuesInterested, *valuesReference = NULL;
uint64_t tmp;
bool uniqueInterested = ui->ckUniqueToInterested->isChecked();
+24 -21
View File
@@ -391,27 +391,30 @@ void FrameSenderWindow::doModifiers(int idx)
second = fetchOperand(idx, op.second);
switch (op.operation)
{
case ADDITION:
shadowReg = first + second;
break;
case AND:
shadowReg = first & second;
break;
case DIVISION:
shadowReg = first / second;
break;
case MULTIPLICATION:
shadowReg = first * second;
break;
case OR:
shadowReg = first | second;
break;
case SUBTRACTION:
shadowReg = first - second;
break;
case XOR:
shadowReg = first ^ second;
break;
case ADDITION:
shadowReg = first + second;
break;
case AND:
shadowReg = first & second;
break;
case DIVISION:
shadowReg = first / second;
break;
case MULTIPLICATION:
shadowReg = first * second;
break;
case OR:
shadowReg = first | second;
break;
case SUBTRACTION:
shadowReg = first - second;
break;
case XOR:
shadowReg = first ^ second;
break;
case NOT:
shadowReg = !shadowReg;
break;
}
}
//Finally, drop the result into the proper data byte
+13
View File
@@ -395,6 +395,19 @@ void SerialWorker::procRXChar(unsigned char c)
}
rx_step++;
break;
case TIME_SYNC:
rx_state = IDLE;
break;
case SET_DIG_OUTPUTS:
rx_state = IDLE;
break;
case SETUP_CANBUS:
rx_state = IDLE;
break;
case SET_SINGLEWIRE_MODE:
rx_state = IDLE;
break;
}
}