EPICS Controls Argonne National Laboratory

Experimental Physics and
Industrial Control System

2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  <20162017  2018  2019  2020  2021  2022  Index 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  <20162017  2018  2019  2020  2021  2022 
<== Date ==> <== Thread ==>

Subject: Re: Merge conflict in base-3.16
From: Michael Davidsaver <[email protected]>
To: [email protected]
Date: Tue, 6 Sep 2016 19:31:48 -0400
On 09/06/2016 07:04 PM, Andrew Johnson wrote:
> While merging the 3.15 changes into 3.16 I discovered these conflicting
> changes. Which one is correct, and if it's the 3.16 change, should it be
> back-ported to 3.15?

I think yours, though it may not make a difference practically.  If
copyInHeader() only fails in allocRawMsg() fails, in which case I think
no new message is ready to be sent.  In this case I would think that
commitMsg() is either a no-op, or an error.  I haven't look far enough
to tell which is the case.

> Whichever is right, I'm almost done with updating the 3.16 branch.


On 09/06/2016 07:04 PM, Andrew Johnson wrote:
> On the 3.15 branch, Ralph committed this change:
> 
>> ------------------------------------------------------------
>> revno: 12770
>> committer: Ralph Lange <[email protected]>
>> branch nick: trunk
>> timestamp: Thu 2016-05-26 14:14:56 +0200
>> message:
>>   pcas: fix compiler warning (gcc 5.3.1)
>> diff:
>> === modified file 'src/ca/legacy/pcas/generic/casDGClient.cc'
>> --- src/ca/legacy/pcas/generic/casDGClient.cc   2012-04-12 16:13:50 +0000
>> +++ src/ca/legacy/pcas/generic/casDGClient.cc   2016-05-26 12:14:56 +0000
>> @@ -348,10 +348,8 @@
>>  //
>>  caStatus casDGClient::searchFailResponse ( const caHdrLargeArray * mp )
>>  {
>> -       int             status;
>> -
>>      epicsGuard < epicsMutex > guard ( this->mutex );
>> -    status = this->out.copyInHeader ( CA_PROTO_NOT_FOUND, 0,
>> +    this->out.copyInHeader ( CA_PROTO_NOT_FOUND, 0,
>>          mp->m_dataType, mp->m_count, mp->m_cid, mp->m_available, 0 );
>>  
>>         this->out.commitMsg ();
> 
> 
> On the 3.16 branch, I committed this change:
> 
>> ------------------------------------------------------------
>> revno: 12695
>> committer: Andrew Johnson <[email protected]>
>> branch nick: 3.16
>> timestamp: Sun 2015-10-18 21:41:46 -0500
>> message:
>>   Clean up g++ unused-but-set-variable warning in pcas
>> diff:
>> === modified file 'src/ca/legacy/pcas/generic/casDGClient.cc'
>> --- src/ca/legacy/pcas/generic/casDGClient.cc   2012-04-12 16:13:50 +0000
>> +++ src/ca/legacy/pcas/generic/casDGClient.cc   2015-10-19 02:41:46 +0000
>> @@ -354,9 +354,11 @@
>>      status = this->out.copyInHeader ( CA_PROTO_NOT_FOUND, 0,
>>          mp->m_dataType, mp->m_count, mp->m_cid, mp->m_available, 0 );
>>  
>> -       this->out.commitMsg ();
>> +    if ( status == S_cas_success ) {
>> +        this->out.commitMsg ();
>> +    }
>>  
>> -       return S_cas_success;
>> +    return S_cas_success;
>>  }
>>  
>>  /*
> 
> While merging the 3.15 changes into 3.16 I discovered these conflicting
> changes. Which one is correct, and if it's the 3.16 change, should it be
> back-ported to 3.15?
> 
> Whichever is right, I'm almost done with updating the 3.16 branch.
> 
> - Andrew
> 


Replies:
Re: Merge conflict in base-3.16 Ralph Lange
References:
Merge conflict in base-3.16 Andrew Johnson

Navigate by Date:
Prev: Merge conflict in base-3.16 Andrew Johnson
Next: Fix for building 3.15.4 on windows-x64-static Mark Rivers
Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  <20162017  2018  2019  2020  2021  2022 
Navigate by Thread:
Prev: Merge conflict in base-3.16 Andrew Johnson
Next: Re: Merge conflict in base-3.16 Ralph Lange
Index: 2002  2003  2004  2005  2006  2007  2008  2009  2010  2011  2012  2013  2014  2015  <20162017  2018  2019  2020  2021  2022 
ANJ, 07 Sep 2016 Valid HTML 4.01! · Home · News · About · Base · Modules · Extensions · Distributions · Download ·
· Search · EPICS V4 · IRMIS · Talk · Bugs · Documents · Links · Licensing ·