Environment.TickCount in OnStart about quickfixn HOT 9 CLOSED

Great catch. Is there a chance you can attach your altered code as a
starting point?

Better yet, we always welcome contributor patches (make a fork and fix
it, then submit a pull request), if you're open to giving it a shot.

from quickfixn.

// instead of lastConnectTickCount
private DateTime lastConnectTime = DateTime.MinValue;

.
.
.

protected override void OnStart()
{
shutdownRequested_ = false;

        while(!shutdownRequested_)
        {
            DateTime now = DateTime.Now;
            if (now.Subtract(lastConnectTime).Seconds >= reconnectInterval_)
            {
                Connect();
                lastConnectTime = now;
            }

            Thread.Sleep(1 * 1000);
        }
    }

from quickfixn.

We will need to fix the test request timer and the heartbeat timer, which also use TickCount.

from quickfixn.

A different code solution was pasted in #53.

Lots of people seem to be hitting this, so we should probably try to make some fix soon.

from quickfixn.

I'll give it a shot.

from quickfixn.

Issue still open. #55's submission was rejected because tests didn't build.

from quickfixn.

Sorry for the delay Grant. Had a bunch of other stuff come up at
work. I hope to have this committed with fixes to the unit tests etc
by the end of the day.

On Wed, Apr 25, 2012 at 2:48 PM, Grant Birchmeier
[email protected]
wrote:

Issue still open.  #55's submission was rejected because tests didn't build.

---

Reply to this email directly or view it on GitHub:
#42 (comment)

from quickfixn.

I just pulled mjwood7's fix into master. Can we get confirmation that this resolves the issue?

from quickfixn.

Closed by pull request #63.

from quickfixn.