A collegue of mine has pointed out an issue in OrderListAdd() / ADSIndex.prg:
If the result of ACE.ADSOpenIndex() is ACE.AE_INDEX_ALREADY_OPEN (5065) in line 481 the logic checks all open indices against the desired one. If there is a hit self:Index is set correctly and the surrounding logic could proceed w/o problems.
However since there ist still 5065 in result, SELF:_CheckError(result,EG_OPEN) in line 506 throws an error which leads to confusing protocols. Would it make sense to set result back to AE_SUCCESS in line 491 before exiting the fallback logic?
I have attached a screenshot of our proposal (blue line). Feel free to comment
Possible improvement for OrderListAdd in XSharp.RDD
-
- Posts: 71
- Joined: Thu Jul 15, 2021 10:46 am
- Location: Germany
Possible improvement for OrderListAdd in XSharp.RDD
Stefan,
Good catch!
Robert
Good catch!
Robert
XSharp Development Team
The Netherlands
robert@xsharp.eu
The Netherlands
robert@xsharp.eu