Re: [Quickfix-developers] QuickFix 64 bit port

[ciju j. <jo...@oc...>]

On Wed, Dec 3, 2008 at 5:43 PM, Wilhelm Thomas <th...@cu...> wrote:
> Hi Ciju
>
> Thanks - Sorry for the delay, I was off for few days.
> I just try your code and it works well. This is great.

Are you using a 64 bit QF build?

We are seeing good performance on our end as well. Thanks for
validating the changes.

>
> 1) I'm using c#. The only issue I can see but this is not from your code
> (this issue was there before) is a deadlock, it occurs if there is a
> connection delay or drop out. An easy way to simulate a connection drop is
> to unplug the network and plug it again or disable the "network connection"
> and enable it again, then QuickFix will hang in the c ++ code. After this I
> can not reconnect the initiator (ThreadedSocketInitiator) and I need to
> crash the c# application and start again, are you experiencing the same
> thing?

No, I haven't run into this. I expect someone else may have seen this
as this looks like an often encountered scenario.

>
> 2) also when I run some c# application in debug mode, sometime I see this
>
>
> Any idea what is it and how to fix this?

No ideas there.

>
> 3) When will you push your modifications in svn?

I haven't heard back from the QF administrators on this issue. What I
published are my final changes, so I will be happy if someone with
commit access pushes this into the repo.

>
> I'll run more tests for the c# version

thanks. Keep me posted.

Ciju

>
> Thanks again
>
> Wilhelm
>
>
> Attached is a zip with just the modified files. Unzip in local
> workspace will update relevant files.
>
> thanks,
> Ciju
>
> On Mon, Nov 24, 2008 at 10:25 AM, ciju john <jo...@oc...> wrote:
>
>
> Hi all.
>
> I have been working on a x64 port of QuickFix. The route we took was
> to add new debug/release build configurations to the VC++ 8 project.
> We have had to recreate a missing file
>
> src/.NET/FieldMap.cpp
>
> to get everything built. Attached is a diff of my changes. I would be
> very thankful for a review of the changes and would like to push this
> into the repo.
>
> thanks,
> Ciju
>
>