mirror of
https://github.com/arduino/Arduino.git
synced 2024-12-03 14:24:15 +01:00
3035239a4e
Previously, pointer casting was used, but this resulted in strict-aliasing warnings:
IPAddress.h: In member function ‘IPAddress::operator uint32_t() const’:
IPAddress.h:46:61: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
operator uint32_t() const { return *((uint32_t*)_address); };
^
IPAddress.h: In member function ‘bool IPAddress::operator==(const IPAddress&) const’:
IPAddress.h:47:81: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
bool operator==(const IPAddress& addr) const { return (*((uint32_t*)_address)) == (*((uint32_t*)addr._address)); };
^
IPAddress.h:47:114: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing]
bool operator==(const IPAddress& addr) const { return (*((uint32_t*)_address)) == (*((uint32_t*)addr._address)); };
Converting between unrelated types like this is commonly done using a union,
which do not break the strict-aliasing rules. Using that union, inside
IPAddress there is now an attribute _address.bytes for the raw byte
arra, or _address.dword for the uint32_t version.
Since we now have easy access to the uint32_t version, this also removes
two memcpy invocations that can just become assignments.
This patch does not change the generated code in any way, the compiler
already optimized away the memcpy calls and the previous casts mean
exactly the same.
This is a different implementation of a part of #1399 and it helps
toward fixing #1728.
75 lines
1.9 KiB
C++
75 lines
1.9 KiB
C++
/*
|
|
IPAddress.cpp - Base class that provides IPAddress
|
|
Copyright (c) 2011 Adrian McEwen. All right reserved.
|
|
|
|
This library is free software; you can redistribute it and/or
|
|
modify it under the terms of the GNU Lesser General Public
|
|
License as published by the Free Software Foundation; either
|
|
version 2.1 of the License, or (at your option) any later version.
|
|
|
|
This library is distributed in the hope that it will be useful,
|
|
but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
Lesser General Public License for more details.
|
|
|
|
You should have received a copy of the GNU Lesser General Public
|
|
License along with this library; if not, write to the Free Software
|
|
Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA
|
|
*/
|
|
|
|
#include <Arduino.h>
|
|
#include <IPAddress.h>
|
|
|
|
IPAddress::IPAddress()
|
|
{
|
|
_address.dword = 0;
|
|
}
|
|
|
|
IPAddress::IPAddress(uint8_t first_octet, uint8_t second_octet, uint8_t third_octet, uint8_t fourth_octet)
|
|
{
|
|
_address.bytes[0] = first_octet;
|
|
_address.bytes[1] = second_octet;
|
|
_address.bytes[2] = third_octet;
|
|
_address.bytes[3] = fourth_octet;
|
|
}
|
|
|
|
IPAddress::IPAddress(uint32_t address)
|
|
{
|
|
_address.dword = address;
|
|
}
|
|
|
|
IPAddress::IPAddress(const uint8_t *address)
|
|
{
|
|
memcpy(_address.bytes, address, sizeof(_address.bytes));
|
|
}
|
|
|
|
IPAddress& IPAddress::operator=(const uint8_t *address)
|
|
{
|
|
memcpy(_address.bytes, address, sizeof(_address.bytes));
|
|
return *this;
|
|
}
|
|
|
|
IPAddress& IPAddress::operator=(uint32_t address)
|
|
{
|
|
_address.dword = address;
|
|
return *this;
|
|
}
|
|
|
|
bool IPAddress::operator==(const uint8_t* addr) const
|
|
{
|
|
return memcmp(addr, _address.bytes, sizeof(_address.bytes)) == 0;
|
|
}
|
|
|
|
size_t IPAddress::printTo(Print& p) const
|
|
{
|
|
size_t n = 0;
|
|
for (int i =0; i < 3; i++)
|
|
{
|
|
n += p.print(_address.bytes[i], DEC);
|
|
n += p.print('.');
|
|
}
|
|
n += p.print(_address.bytes[3], DEC);
|
|
return n;
|
|
}
|
|
|