>> Yep, it looks much better. I found it strange that the later did a reset but
>> was not doing the set.
>>
>> Attached v2 does as you suggest.
>
> Close enough. I was thinking about this position of the attached,
> which is more consistent with the rest.
Given the structural complexity of the function, the end of the file
seemed like a good place to have an all-path-guaranteed reset.
I find it a little bit strange to have the Set at the upper level and the
Reset in many… but not all branches, though.
For instance the on_error_rollback_savepoint/svptcmd branch includes a
reset long after many other conditional resets, I cannot guess whether the
initial set is still active or has been long wiped out and this query is
just not cancellable.
Also, ISTM that in the worst case a cancellation request is sent to a
server which is idle, in which case it will be ignored, so the code should
be in no hurry to clean it, at least not at the price of code clarity.
Anyway, the place you suggest seems ok.
--
Fabien.
