EPICS Home

Experimental Physics and Industrial Control System


 
2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  <2017>  2018  2019  2020  2021  2022  2023  2024  Index 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  2016  <2017>  2018  2019  2020  2021  2022  2023  2024 
<== Date ==> <== Thread ==>
Subject: Re: Fwd: bugfix of PCAS library
From: Andrew Johnson <[email protected]>
To: Wang Xiaoqiang <[email protected]>, EPICS core-talk <[email protected]>
Date: Wed, 10 May 2017 10:25:32 -0500 
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