aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
-rw-r--r--lib/ClockTAI.cpp4
-rw-r--r--lib/edi/PFT.cpp8
2 files changed, 6 insertions, 6 deletions
diff --git a/lib/ClockTAI.cpp b/lib/ClockTAI.cpp
index 626140e..bfa7895 100644
--- a/lib/ClockTAI.cpp
+++ b/lib/ClockTAI.cpp
@@ -314,7 +314,7 @@ ClockTAI::ClockTAI(const std::vector<std::string>& bulletin_urls) :
m_bulletin_urls = bulletin_urls;
}
- for (const auto url : m_bulletin_urls) {
+ for (const auto& url : m_bulletin_urls) {
etiLog.level(info) << "TAI Bulletin URL: '" << url << "'";
}
}
@@ -359,7 +359,7 @@ int ClockTAI::get_valid_offset()
#endif
}
else {
- for (const auto url : m_bulletin_urls) {
+ for (const auto& url : m_bulletin_urls) {
try {
#if TAI_TEST
etiLog.level(info) << "Load bulletin from " << url;
diff --git a/lib/edi/PFT.cpp b/lib/edi/PFT.cpp
index 25f2d1f..2877aa5 100644
--- a/lib/edi/PFT.cpp
+++ b/lib/edi/PFT.cpp
@@ -453,11 +453,11 @@ std::vector<uint8_t> AFBuilder::extractAF()
}
// EDI specific, must have a CRC.
- if( _af_packet.size() >= 12 ) {
+ if (_af_packet.size() >= 12) {
ok = checkCRC(_af_packet.data(), _af_packet.size());
if (not ok) {
- etiLog.log(debug, "Too many errors to reconstruct AF from %zu/%u"
+ etiLog.log(debug, "CRC error after AF reconstruction from %zu/%u"
" PFT fragments\n", _fragments.size(), _Fcount);
}
}
@@ -570,7 +570,7 @@ afpacket_pft_t PFT::getNextAFPacket()
if (builder.canAttemptToDecode() == dar_t::yes) {
auto afpacket = builder.extractAF();
- assert(not afpacket.empty());
+ // Empty AF Packet can happen if CRC is wrong
if (m_verbose) {
etiLog.level(debug) << "Fragment origin stats: " << builder.visualise_fragment_origins();
}
@@ -588,7 +588,7 @@ afpacket_pft_t PFT::getNextAFPacket()
auto afpacket = builder.extractAF();
if (afpacket.empty()) {
- etiLog.log(debug,"pseq %d timed out after RS", m_next_pseq);
+ etiLog.log(debug, "pseq %d timed out after RS", m_next_pseq);
}
if (m_verbose) {
etiLog.level(debug) << "Fragment origin stats: " << builder.visualise_fragment_origins();