Allows undo maritime trade. I dug deep into the state machine for this, the only way to learn it...
The one thing I missed was to place the bank trade back into the trade quote, but for now, just refresh it by deselecting and reselecting a resource check box.
While I was using this patch, the server crashed on me during a 4:1 trade, I have no log, I need to troubleshoot it.
Fixed the segfault bug and added V0_13 in ClientVersionType to prevent an error message with older clients.
allows undo maritime trade. version 3
I've updated the patch to match the current code (svn revision 1946)
I think this feature is nice to have in the Pioneers 15 release.
I would propose to change the following (assign the issue to me to let me make them):
Go ahead and make the changes. I was thinking about addressing this patch after my current thread, but what will be awhile. Just interesting how time works. I had more time to work on things in the past, while you where busy. Now it seems to be the other way.
As mailed to pio-develop on 2013-09-02
MB For patch 614 undo maritime trade, you said “I think this feature is
MB nice to have in the Pioneers 15 release.” Are you going to add this
MB patch in the release?
RC I was working on it. But I've found that the undo system has some subtle
RC bugs, unexpected behaviours/code paths, doesn't follow the master/slave
RC model and then I got side-tracked.
Therefore I unassign the patch. I still think this would a good feature, but it will not make it for V15.