IMessageStore extended with method SetAndIncrNextSenderMsgSeqNum, resolves issue #987

[asmeisne]

The IMessageStore interface is extended with a new method SetAndIncrNextSenderMsgSeqNum with a default implementation to ensure backwards compatibility. This new method now is used in Session.Persist() to allow execution of methods IMessageStore.Set() and IMessageStore.IncrNextSenderMsgSeqNum() in one transaction. Two unit tests for FileStore and MemoryStore are added.

resolves #987

[CLAassistant]

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ gbirchmeier
❌ ameisner-boerse
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

[gbirchmeier]

@asmeisne Can you please click this link to sign the CLA? Thanks!

https://cla-assistant.io/connamara/quickfixn?pullRequest=993

[gbirchmeier]

I rebased to latest master, moved the functions to be closer to the others, and changed the doc-comment a little. I will merge as soon as you accept the CLA!

[mirkosowalabs]

@gbirchmeier how can we move this pull request forward from @asmeisne ?

[gbirchmeier]

@mirkosowalabs It took me like 3 months to act on asmeisne's report. I think it's fair to give him more than 2 days to respond.

[asmeisne]

@asmeisne Can you please click this link to sign the CLA? Thanks!

https://cla-assistant.io/connamara/quickfixn?pullRequest=993

@gbirchmeier I already signed the CLA as asmeisne but the comment above shows that signing is required by ameisner-boerse. Since I am no more member of boerse-stuttgart, I do not have access to this account anymore. Can you fix this?

[gbirchmeier]

Thanks for the explanation, that seems sufficient!