luke
/
prjxray
mirror of https://github.com/openXC7/prjxray.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

lib: xc7series: const all the Span<>s 

Span<T> can be converted to a Span<const T> but the reverse is not true.
Since most of the uses of spans do not allow modification of the data,
taking a Span<const T> is more versatile and importantly allows them to
be used in const methods.

Signed-off-by: Rick Altherr <kc8apf@kc8apf.net>
This commit is contained in:
Rick Altherr 2018-01-18 20:00:28 -08:00
parent 84d09c833e
commit b0604362e0
3 changed files with 7 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
lib/include/prjxray/xilinx/xc7series/bitstream_writer.h
View File

@ -25,8 +25,8 @@ class BitstreamWriter {
typedef std::array<uint32_t, 6> header_t;
typedef std::vector<ConfigurationPacket> packets_t;
// Only defined if a packet exists
typedef absl::optional<absl::Span<uint32_t>> op_data_t;
typedef absl::Span<uint32_t>::iterator data_iterator_t;
typedef absl::optional<absl::Span<const uint32_t>> op_data_t;
typedef absl::Span<const uint32_t>::iterator data_iterator_t;
using itr_value_type = uint32_t;
class packet_iterator

4
lib/include/prjxray/xilinx/xc7series/configuration.h
View File

@ -16,7 +16,7 @@ namespace xc7series {
class Configuration {
public:
using FrameMap = std::map<FrameAddress, absl::Span<uint32_t>>;
using FrameMap = std::map<FrameAddress, absl::Span<const uint32_t>>;
template <typename Collection>
static absl::optional<Configuration> InitWithPackets(
@ -28,7 +28,7 @@ class Configuration {
: part_(part) {
for (auto& frame : *frames) {
frames_[frame.first] =
absl::Span<uint32_t>(frame.second);
absl::Span<const uint32_t>(frame.second);
}
}

6
lib/include/prjxray/xilinx/xc7series/configuration_packet.h
View File

@ -27,7 +27,7 @@ class ConfigurationPacket {
ConfigurationPacket(unsigned int header_type,
Opcode opcode,
ConfigurationRegister address,
const absl::Span<uint32_t>& data)
const absl::Span<const uint32_t>& data)
: header_type_(header_type),
opcode_(opcode),
address_(address),
@ -47,13 +47,13 @@ class ConfigurationPacket {
unsigned int header_type() const { return header_type_; }
const Opcode opcode() const { return opcode_; }
const ConfigurationRegister address() const { return address_; }
const absl::Span<uint32_t>& data() const { return data_; }
const absl::Span<const uint32_t>& data() const { return data_; }
private:
unsigned int header_type_;
Opcode opcode_;
ConfigurationRegister address_;
absl::Span<uint32_t> data_;
absl::Span<const uint32_t> data_;
};
std::ostream& operator<<(std::ostream& o, const ConfigurationPacket& packet);