Re: base-3.16.1 bug

[Andrew Johnson <anj@aps.anl.gov>]

Fix and new tests committed to the Base-3.16 branch.

- Andrew

On 09/06/2017 06:17 PM, Andrew Johnson wrote:
> On 09/06/2017 12:12 PM, Michael Davidsaver wrote:
>> @Andrew Will you be adding a test to trigger this regression before
>> fixing it?  src/std/rec/test/regressTest.c would be a easy place.
> 
> I had already started creating a new softTest program, analogous to the
> asyncSoftTest code already under std/rec/test. Will finish off tomorrow.
> 
>>> I would say the dummy method returning OK is the way to go. Other link
>>> types might return errors that are worth reporting, and it is certainly
>>> a property of the CONSTANT link that a put() operation is never
>>> producing an error.
>>
>> I agree with this.
> 
> Thanks for responding.
> 
>> Another inconsistency.  CA put w/ callback is losing the error.  I see
>> that dbNotify ignores the result of dbProcess().  It would be nice if it
>> didn't...
> 
> That's a whole other level of complexity, the result would have to be
> saved in the processNotify object by dbNotify and then retrieved and
> returned to the client by the rsrv callback routine — the first part is
> easy, but the second isn't obvious...
> 
>> I also noticed that dbNotify doesn't propagate some errors in it's
>> putCallback for link fields.
> 
> Similar, I think that's a different bug (and I shouldn't really have
> been spending time writing tests for the original problem at the moment,
> but I did).
> 
> - Andrew
> 

-- 
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