Skip to content

Commit 66d033d

Browse files
committed
LteMac*: rename confusing local variable (header -> macPdu)
1 parent c98cf5f commit 66d033d

4 files changed

Lines changed: 18 additions & 19 deletions

File tree

src/simu5g/stack/mac/LteMacEnb.cc

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -599,7 +599,6 @@ void LteMacEnb::macPduMake(MacCid cid)
599599
UnitList txList = (txBuf->firstAvailable());
600600

601601
auto macPacket = pit.second;
602-
auto header = macPacket->peekAtFront<LteMacPdu>();
603602
EV << "LteMacBase: PDU Maker created PDU: " << macPacket->str() << endl;
604603

605604
// PDU transmission here (if any)

src/simu5g/stack/mac/LteMacUe.cc

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -415,9 +415,9 @@ void LteMacUe::macPduMake(MacCid cid) //TODO note cid is unused here!!!!
415415
auto pkt = check_and_cast<Packet *>(mbuf_[destCid]->popFront());
416416
drop(pkt);
417417

418-
auto header = macPkt->removeAtFront<LteMacPdu>();
419-
header->pushSdu(pkt);
420-
macPkt->insertAtFront(header);
418+
auto macPdu = macPkt->removeAtFront<LteMacPdu>();
419+
macPdu->pushSdu(pkt);
420+
macPkt->insertAtFront(macPdu);
421421
sduPerCid--;
422422
}
423423
// consider virtual buffers to compute BSR size
@@ -533,13 +533,13 @@ void LteMacUe::macPduMake(MacCid cid) //TODO note cid is unused here!!!!
533533
// }
534534

535535
bool bsrAlreadyMade = false;
536-
auto header = macPkt->removeAtFront<LteMacPdu>();
536+
auto macPdu = macPkt->removeAtFront<LteMacPdu>();
537537
if (bsrTriggered_) {
538538
MacBsr *bsr = new MacBsr();
539539

540540
bsr->setTimestamp(simTime().dbl());
541541
bsr->setSize(size);
542-
header->pushCe(bsr);
542+
macPdu->pushCe(bsr);
543543

544544
bsrTriggered_ = false;
545545
bsrAlreadyMade = true;
@@ -553,7 +553,7 @@ void LteMacUe::macPduMake(MacCid cid) //TODO note cid is unused here!!!!
553553
bsrRtxTimer_ = 0;
554554

555555
// insert updated MacPdu
556-
macPkt->insertAtFront(header);
556+
macPkt->insertAtFront(macPdu);
557557

558558
EV << "LteMacUe: pduMaker created PDU: " << macPkt->str() << endl;
559559

src/simu5g/stack/mac/LteMacUeD2D.cc

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -243,9 +243,9 @@ void LteMacUeD2D::macPduMake(MacCid cid)
243243

244244
drop(pkt);
245245

246-
auto header = macPkt->removeAtFront<LteMacPdu>();
247-
header->pushSdu(pkt);
248-
macPkt->insertAtFront(header);
246+
auto macPdu = macPkt->removeAtFront<LteMacPdu>();
247+
macPdu->pushSdu(pkt);
248+
macPkt->insertAtFront(macPdu);
249249
sduPerCid--;
250250
}
251251

@@ -375,13 +375,13 @@ void LteMacUeD2D::macPduMake(MacCid cid)
375375
//
376376
// }
377377

378-
auto header = macPkt->removeAtFront<LteMacPdu>();
378+
auto macPdu = macPkt->removeAtFront<LteMacPdu>();
379379
// Attach BSR to PDU if RAC is won and wasn't already made
380380
if ((bsrTriggered_ || bsrD2DMulticastTriggered_) && !bsrAlreadyMade && size > 0) {
381381
MacBsr *bsr = new MacBsr();
382382
bsr->setTimestamp(simTime().dbl());
383383
bsr->setSize(size);
384-
header->pushCe(bsr);
384+
macPdu->pushCe(bsr);
385385
bsrTriggered_ = false;
386386
bsrD2DMulticastTriggered_ = false;
387387
bsrAlreadyMade = true;
@@ -393,7 +393,7 @@ void LteMacUeD2D::macPduMake(MacCid cid)
393393
else
394394
bsrRtxTimer_ = 0;
395395

396-
macPkt->insertAtFront(header);
396+
macPkt->insertAtFront(macPdu);
397397

398398
EV << "LteMacUeD2D: pduMaker created PDU: " << macPkt->str() << endl;
399399

src/simu5g/stack/mac/NrMacUe.cc

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -449,9 +449,9 @@ void NrMacUe::macPduMake(MacCid cid)
449449

450450
drop(pkt);
451451

452-
auto header = macPkt->removeAtFront<LteMacPdu>();
453-
header->pushSdu(pkt);
454-
macPkt->insertAtFront(header);
452+
auto macPdu = macPkt->removeAtFront<LteMacPdu>();
453+
macPdu->pushSdu(pkt);
454+
macPkt->insertAtFront(macPdu);
455455
sduPerCid--;
456456
}
457457

@@ -581,13 +581,13 @@ void NrMacUe::macPduMake(MacCid cid)
581581
//
582582
// } */
583583

584-
auto header = macPkt->removeAtFront<LteMacPdu>();
584+
auto macPdu = macPkt->removeAtFront<LteMacPdu>();
585585
// Attach BSR to PDU if RAC is won and wasn't already made
586586
if ((bsrTriggered_ || bsrD2DMulticastTriggered_) && !bsrAlreadyMade && size > 0) {
587587
MacBsr *bsr = new MacBsr();
588588
bsr->setTimestamp(simTime().dbl());
589589
bsr->setSize(size);
590-
header->pushCe(bsr);
590+
macPdu->pushCe(bsr);
591591
bsrTriggered_ = false;
592592
bsrD2DMulticastTriggered_ = false;
593593
bsrAlreadyMade = true;
@@ -600,7 +600,7 @@ void NrMacUe::macPduMake(MacCid cid)
600600
else
601601
bsrRtxTimer_ = 0;
602602

603-
macPkt->insertAtFront(header);
603+
macPkt->insertAtFront(macPdu);
604604

605605
EV << "NrMacUe: pduMaker created PDU: " << macPkt->str() << endl;
606606

0 commit comments

Comments
 (0)