Merged, thanks all!
- Andrew
On 05/10/2017 05:00 AM, Wang Xiaoqiang wrote:
> I agree this is a quick fix by duplicating code and refactoring is needed.
>
> One idea I had was to add an API virtual function "getValueSizeElements"
> to gdd, in analog to "getDataSizeElements". But that would make gdd
> interface depend on gddAppTable, which might be a bad practice.
>
> On 05/10/2017 12:57 AM, 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.
>>
>> Thanks Xiaoqiang!
>>
>> - Andrew
>>
>> -------- Forwarded Message --------
>> Subject: bugfix of PCAS library
>> Date: Fri, 5 May 2017 13:21:50 +0200
>> From: Wang Xiaoqiang <[email protected]>
>> To: tech-talk <[email protected]>
>>
>>
>> Hi,
>>
>> The problem was discovered using a PCASpy server application.
>> https://github.com/paulscherrerinstitute/pcaspy/issues/43
>>
>> A short summary is that the pcas library calculate the wrong size if a
>> container type gdd is returned by a server application, in a read action
>> with zero requested count.
>>
>> The proposed change has been verified to fix the issue.
>> https://github.com/xiaoqiangwang/epics-base/commit/f1367b28c6e620f2b4abb276812df854c47e3a48
>>
>>
>> Best
>> Xiaoqiang
>>
--
Arguing for surveillance because you have nothing to hide is no
different than making the claim, "I don't care about freedom of
speech because I have nothing to say." -- Edward Snowdon
- References:
- Fwd: bugfix of PCAS library Andrew Johnson
- Re: Fwd: bugfix of PCAS library Wang Xiaoqiang
- Navigate by Date:
- Prev:
Re: Fwd: bugfix of PCAS library Wang Xiaoqiang
- Next:
Build failed in Jenkins: epics-base-3.15-win64-test #108 APS Jenkins
- 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 Wang Xiaoqiang
- Next:
Build failed in Jenkins: epics-base-3.15-win64-test #108 APS Jenkins
- Index:
2002
2003
2004
2005
2006
2007
2008
2009
2010
2011
2012
2013
2014
2015
2016
<2017>
2018
2019
2020
2021
2022
2023
2024
|