Skip to content

Commit 9a5d9f0

Browse files
committed
report underflows in debug mode only;
1 parent 0decd05 commit 9a5d9f0

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

src/host/modem/Modem.cpp

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1413,7 +1413,8 @@ bool Modem::writeDMRFrame1(const uint8_t* data, uint32_t length, bool imm)
14131413

14141414
m_dmrSpace1 -= length;
14151415
if ((int32_t)m_dmrSpace1 < 0U) {
1416-
LogError(LOG_MODEM, "Modem::writeDMRFrame1(), dmrSpace1 underflow, space = %u, length = %u", m_dmrSpace1, length);
1416+
if (m_debug)
1417+
LogError(LOG_MODEM, "Modem::writeDMRFrame1()", "dmrSpace1 underflow, space = %u, length = %u", m_dmrSpace1, length);
14171418
m_dmrSpace1 = 0U;
14181419
}
14191420
}
@@ -1471,7 +1472,8 @@ bool Modem::writeDMRFrame2(const uint8_t* data, uint32_t length, bool imm)
14711472

14721473
m_dmrSpace2 -= length;
14731474
if ((int32_t)m_dmrSpace2 < 0U) {
1474-
LogError(LOG_MODEM, "Modem::writeDMRFrame2(), dmrSpace2 underflow, space = %u, length = %u", m_dmrSpace2, length);
1475+
if (m_debug)
1476+
LogDebugEx(LOG_MODEM, "Modem::writeDMRFrame2()", "dmrSpace2 underflow, space = %u, length = %u", m_dmrSpace2, length);
14751477
m_dmrSpace2 = 0U;
14761478
}
14771479
}
@@ -1542,7 +1544,8 @@ bool Modem::writeP25Frame(const uint8_t* data, uint32_t length, bool imm)
15421544

15431545
m_p25Space -= length;
15441546
if ((int32_t)m_p25Space < 0U) {
1545-
LogError(LOG_MODEM, "Modem::writeP25Frame(), p25Space underflow, space = %u, length = %u", m_p25Space, length);
1547+
if (m_debug)
1548+
LogDebugEx(LOG_MODEM, "Modem::writeP25Frame()", "p25Space underflow, space = %u, length = %u", m_p25Space, length);
15461549
m_p25Space = 0U;
15471550
}
15481551
}
@@ -1600,7 +1603,8 @@ bool Modem::writeNXDNFrame(const uint8_t* data, uint32_t length, bool imm)
16001603

16011604
m_nxdnSpace -= length;
16021605
if ((int32_t)m_nxdnSpace < 0U) {
1603-
LogError(LOG_MODEM, "Modem::writeNXDNFrame(), nxdnSpace underflow, space = %u, length = %u", m_nxdnSpace, length);
1606+
if (m_debug)
1607+
LogDebugEx(LOG_MODEM, "Modem::writeNXDNFrame()", "nxdnSpace underflow, space = %u, length = %u", m_nxdnSpace, length);
16041608
m_nxdnSpace = 0U;
16051609
}
16061610
}

0 commit comments

Comments
 (0)