Looks appropriate to me as well. Would like to see this in 3.14.
Cheers,
- Bruce
On 5/9/2017 4:30 PM, Michael Davidsaver wrote:
On 05/09/2017 06:57 PM, Andrew Johnson wrote:
Unless anyone objects I propose to cherry-pick this commit onto the 3.14
branch. As Xiaoqiang says in the commit message this change has already
been implemented in the monitorResponse methods; this adds the same code
to the readResponse and readNotifyResponse methods, and while I don't
claim to understand the PCAS code in detail I can see that the same
change is likely to be needed there for the PCAS to support dynamic
arrays properly.
This change looks as reasonable as any involving GDD. Use the value
size when meta-data is present.
- References:
- Fwd: bugfix of PCAS library Andrew Johnson
- Re: Fwd: bugfix of PCAS library Michael Davidsaver
- Navigate by Date:
- Prev:
Re: Fwd: bugfix of PCAS library Michael Davidsaver
- Next:
Re: Fwd: bugfix of PCAS library Wang Xiaoqiang
- Index:
2002
2003
2004
2005
2006
2007
2008
2009
2010
2011
2012
2013
2014
2015
2016
<2017>
2018
2019
2020
2021
2022
2023
2024
- Navigate by Thread:
- Prev:
Re: Fwd: bugfix of PCAS library Michael Davidsaver
- Next:
Re: Fwd: bugfix of PCAS library Wang Xiaoqiang
- Index:
2002
2003
2004
2005
2006
2007
2008
2009
2010
2011
2012
2013
2014
2015
2016
<2017>
2018
2019
2020
2021
2022
2023
2024
|