Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(3005)

Unified Diff: src/lte/model/epc-gtpu-header.cc

Issue 338840043: Eliminate Visual Studio compiler warnings (Closed)
Patch Set: Patch updates for x64 build Created 6 years, 2 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/lte/model/epc-gtpu-header.h ('k') | src/lte/model/epc-mme.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/lte/model/epc-gtpu-header.cc
===================================================================
--- a/src/lte/model/epc-gtpu-header.cc
+++ b/src/lte/model/epc-gtpu-header.cc
@@ -77,7 +77,7 @@
{
Buffer::Iterator i = start;
uint8_t firstByte = m_version << 5 | m_protocolType << 4 | 0x1 << 3;
- firstByte |= m_extensionHeaderFlag << 2 | m_sequenceNumberFlag << 1 | m_nPduNumberFlag;
+ firstByte |= static_cast<uint8_t>(m_extensionHeaderFlag) << 2 | static_cast<uint8_t>(m_sequenceNumberFlag) << 1 | static_cast<uint8_t>(m_nPduNumberFlag);
i.WriteU8 (firstByte);
i.WriteU8 (m_messageType);
i.WriteHtonU16 (m_length);
@@ -197,70 +197,70 @@
}
void
-GtpuHeader::SetExtensionHeaderFlag (bool m_extensionHeaderFlag)
+GtpuHeader::SetExtensionHeaderFlag (bool extensionHeaderFlag)
{
- this->m_extensionHeaderFlag = m_extensionHeaderFlag;
+ this->m_extensionHeaderFlag = extensionHeaderFlag;
}
void
-GtpuHeader::SetLength (uint16_t m_length)
+GtpuHeader::SetLength (uint16_t length)
{
- this->m_length = m_length;
+ this->m_length = length;
}
void
-GtpuHeader::SetMessageType (uint8_t m_messageType)
+GtpuHeader::SetMessageType (uint8_t messageType)
{
- this->m_messageType = m_messageType;
+ this->m_messageType = messageType;
}
void
-GtpuHeader::SetNPduNumber (uint8_t m_nPduNumber)
+GtpuHeader::SetNPduNumber (uint8_t nPduNumber)
{
- this->m_nPduNumber = m_nPduNumber;
+ this->m_nPduNumber = nPduNumber;
}
void
-GtpuHeader::SetNPduNumberFlag (bool m_nPduNumberFlag)
+GtpuHeader::SetNPduNumberFlag (bool nPduNumberFlag)
{
- this->m_nPduNumberFlag = m_nPduNumberFlag;
+ this->m_nPduNumberFlag = nPduNumberFlag;
}
void
-GtpuHeader::SetNextExtensionType (uint8_t m_nextExtensionType)
+GtpuHeader::SetNextExtensionType (uint8_t nextExtensionType)
{
- this->m_nextExtensionType = m_nextExtensionType;
+ this->m_nextExtensionType = nextExtensionType;
}
void
-GtpuHeader::SetProtocolType (bool m_protocolType)
+GtpuHeader::SetProtocolType (bool protocolType)
{
- this->m_protocolType = m_protocolType;
+ this->m_protocolType = protocolType;
}
void
-GtpuHeader::SetSequenceNumber (uint16_t m_sequenceNumber)
+GtpuHeader::SetSequenceNumber (uint16_t sequenceNumber)
{
- this->m_sequenceNumber = m_sequenceNumber;
+ this->m_sequenceNumber = sequenceNumber;
}
void
-GtpuHeader::SetSequenceNumberFlag (bool m_sequenceNumberFlag)
+GtpuHeader::SetSequenceNumberFlag (bool sequenceNumberFlag)
{
- this->m_sequenceNumberFlag = m_sequenceNumberFlag;
+ this->m_sequenceNumberFlag = sequenceNumberFlag;
}
void
-GtpuHeader::SetTeid (uint32_t m_teid)
+GtpuHeader::SetTeid (uint32_t teid)
{
- this->m_teid = m_teid;
+ this->m_teid = teid;
}
void
-GtpuHeader::SetVersion (uint8_t m_version)
+GtpuHeader::SetVersion (uint8_t version)
{
// m_version is a uint3_t
- this->m_version = m_version & 0x7;
+ this->m_version = version & 0x7;
}
bool
« no previous file with comments | « src/lte/model/epc-gtpu-header.h ('k') | src/lte/model/epc-mme.cc » ('j') | no next file with comments »

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b