Code Review
/
apps
/
agl-service-can-low-level.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
review
|
tree
raw
|
inline
| side by side
decoder: Fix bit_position swapping
[apps/agl-service-can-low-level.git]
/
low-can-binding
/
can
/
message-definition.cpp
diff --git
a/low-can-binding/can/message-definition.cpp
b/low-can-binding/can/message-definition.cpp
index
064b2c1
..
8060982
100644
(file)
--- a/
low-can-binding/can/message-definition.cpp
+++ b/
low-can-binding/can/message-definition.cpp
@@
-22,16
+22,16
@@
message_definition_t::message_definition_t(
const std::string bus,
uint32_t id,
message_definition_t::message_definition_t(
const std::string bus,
uint32_t id,
-
bool is_fd
,
-
message_format_t format
,
+
uint32_t flags
,
+
bool frame_layout_is_little
,
frequency_clock_t frequency_clock,
bool force_send_changed,
frequency_clock_t frequency_clock,
bool force_send_changed,
- const
std::vector<std::shared_ptr<signal_t> >
& signals)
+ const
vect_ptr_signal_t
& signals)
: parent_{nullptr},
bus_{bus},
id_{id},
: parent_{nullptr},
bus_{bus},
id_{id},
-
is_fd_(is_fd)
,
- f
ormat_{format
},
+
flags_{flags}
,
+ f
rame_layout_is_little_{frame_layout_is_little
},
frequency_clock_{frequency_clock},
force_send_changed_{force_send_changed},
last_value_{CAN_MESSAGE_SIZE},
frequency_clock_{frequency_clock},
force_send_changed_{force_send_changed},
last_value_{CAN_MESSAGE_SIZE},
@@
-42,18
+42,18
@@
message_definition_t::message_definition_t(const std::string bus,
uint32_t id,
const std::string name,
uint32_t length,
uint32_t id,
const std::string name,
uint32_t length,
-
bool is_fd
,
-
message_format_t format
,
+
uint32_t flags
,
+
bool frame_layout_is_little
,
frequency_clock_t frequency_clock,
bool force_send_changed,
frequency_clock_t frequency_clock,
bool force_send_changed,
- const
std::vector<std::shared_ptr<signal_t> >
& signals)
+ const
vect_ptr_signal_t
& signals)
: parent_{nullptr},
bus_{bus},
id_{id},
name_{name},
length_{length},
: parent_{nullptr},
bus_{bus},
id_{id},
name_{name},
length_{length},
-
is_fd_(is_fd)
,
- f
ormat_{format
},
+
flags_{flags}
,
+ f
rame_layout_is_little_{frame_layout_is_little
},
frequency_clock_{frequency_clock},
force_send_changed_{force_send_changed},
last_value_{CAN_MESSAGE_SIZE},
frequency_clock_{frequency_clock},
force_send_changed_{force_send_changed},
last_value_{CAN_MESSAGE_SIZE},
@@
-66,29
+66,33
@@
const std::string message_definition_t::get_bus_device_name() const
.get_can_device_name(bus_);
}
.get_can_device_name(bus_);
}
+const std::string message_definition_t::get_name() const{
+ return name_;
+}
+
uint32_t message_definition_t::get_id() const
{
uint32_t message_definition_t::get_id() const
{
- return id_;
+ return id_ & CAN_EFF_MASK ?
+ id_ | CAN_EFF_FLAG :
+ id_;
}
bool message_definition_t::is_fd() const
{
}
bool message_definition_t::is_fd() const
{
- return
is_fd_
;
+ return
(flags_&FD_FRAME)
;
}
bool message_definition_t::is_j1939() const
{
}
bool message_definition_t::is_j1939() const
{
- if(format_ == message_format_t::J1939)
- {
- return true;
- }
- else
- {
- return false;
- }
+ return (flags_&J1939_PROTOCOL);
+}
+
+bool message_definition_t::is_isotp() const
+{
+ return (flags_&ISOTP_PROTOCOL);
}
}
-
std::vector<std::shared_ptr<signal_t>>
& message_definition_t::get_signals()
+
vect_ptr_signal_t
& message_definition_t::get_signals()
{
return signals_;
}
{
return signals_;
}
@@
-108,7
+112,11
@@
uint32_t message_definition_t::get_length() const
return length_;
}
return length_;
}
-
message_format_t message_definition_t::get_format
() const
+
uint32_t message_definition_t::get_flags
() const
{
{
- return format_;
-}
\ No newline at end of file
+ return flags_;
+}
+
+bool message_definition_t::frame_layout_is_little() const{
+ return frame_layout_is_little_;
+}