<div dir="ltr"><div>Hi all,<br></div><div><br></div><div>So I have some changes coming that depend heavily on this function signature change. What it is doing is taking the members of OvmsVehicle that are specific to the ISOTP (and VWTP) protocols and making the parameters to both IncomingPollReply and IncomingPollError.   </div><div><br></div><div>This way of doing it makes all the bits explicit parameters, which means a bunch more parameters passed on the stack.</div><div><br></div><div><b>Original</b>:</div><div>-    virtual void IncomingPollReply(canbus* bus, uint16_t type, uint16_t pid, const uint8_t* data, uint8_t length, uint16_t mlremain);<br>-    virtual void IncomingPollError(canbus* bus, uint16_t type, uint16_t pid, uint16_t code);<br></div><div><br></div><div><b>New:</b></div><font face="monospace">virtual void IncomingPollReply(canbus* bus, uint32_t moduleidsent, uint32_t moduleid, uint16_t type, uint16_t pid, const uint8_t* data, uint16_t mloffset, uint8_t length, uint16_t mlremain, uint16_t mlframe, const OvmsPoller::poll_pid_t &pollentry);<br>virtual void IncomingPollError(canbus* bus, uint32_t moduleidsent, uint32_t moduleid, uint16_t type, uint16_t pid, uint16_t code, const OvmsPoller::poll_pid_t &pollentry);<br></font><br><div>The other way of achieving the same result would be to create a struct that contains these elements, and then to pass that as a const reference.</div><div><br></div><div>Thoughts?  If I need to make modifications to the signatures, I'd prefer to do it before I make the pull request.</div><div><br></div><div>//.ichael </div></div>