Thread: Re: [Quickfix-developers] dll nightmare
Brought to you by:
orenmnero
From: Miller, O. <OM...@ri...> - 2003-06-17 22:49:31
|
This could be it. In order to run any .NET application, the framework= must be installed. This is not a small thing (on the order of a 100 MB dow= nload).=20 Another thing that you should make sure of is that you are distributin= g the release build. Unlike on *nix, debug libraries and executables re= quire special development libraries to even run. These will not be found on = a typical desktop or production server. -------------------------- Sent from my BlackBerry Wireless Handheld -----Original Message----- From: dwi...@jp... <dwi...@jp...> To: qui...@li... <qui...@li...> Sent: Tue Jun 17 17:21:19 2003 Subject: [Quickfix-developers] dll nightmare I've recently built a release version of quickfix and have numerous people complaining about missing dll's which seem to be exclusive to ms visual studio. Everything works on my development boxes. (big deal) Could it be a missing .NET framework on the end user's desktop? Since I'm a Un*x bigot and very MS ignorant, any suggestions would be helpful. Thanks Dwight Browne 277 Park Ave 46th Floor ------------------------------------------------------- This SF.Net email is sponsored by: INetU Attention Web Developers & Consultants: Become An INetU Hosting Partner. Refer Dedicated Servers. We Manage Them. You Get 10% Monthly Commission! INetU Dedicated Managed Hosting http://www.inetu.net/partner/index.php _______________________________________________ Quickfix-developers mailing list Qui...@li... https://lists.sourceforge.net/lists/listinfo/quickfix-developers ----------------------------------------- This email and any attachments are confidential and may be legally privileged. No confidentiality or privilege is w= aived or lost by any transmission in error. If you are not the intended rec= ipient you are hereby notified that any use, printing, copying or disclosure= is strictly prohibited. Please delete this email and any attachments, with= out printing, copying, forwarding or saving them and notify the sender immed= iately by reply e-mail. The company reserves the right to monitor all e-mai= l communications through its networks. Unless otherwise stated, any financi= al results or price data contained in this email are indicative only and are= subject to change without notice. |
From: Vitor C. <vc...@hi...> - 2003-06-18 09:44:10
|
Hi Oren, I found what I think is a bug introduced during the last optimization done on Session.cpp regarding the use of the toString() methods: In Session::sendRaw has the following code: 432 if ( isLoggedOn() ) result = 433 send( message.toString(messageString) ); ... ... 441 m_state.set( msgSeqNum, messageString ); In which messageString is being changed by the toString method as a side effect. The problem is that it is only changed if the session is logged on and is being used in line 441 to store the message in the messageStore. The effect of this is that if some message is generated while the other party is disconnected, the message doesn't get stored and consequently it will not be resent upon reconnection of the other party. I suggest the following change: Lines 432, 433 should be: 432 messageString = message.toString(); 433 if ( isLoggedOn() ) result = 434 send( messageString ); Best regards, and once again thank you for this excellent piece of code Vitor Castro |
From: Oren M. <ore...@ya...> - 2003-06-18 19:20:05
|
What do you think about instead pulling: messageString = message.toString(); up to line 413, and passing the messageString into send on lines 425 and 433. --- Vitor Castro <vc...@hi...> wrote: > Hi Oren, > > I found what I think is a bug introduced during the > last optimization > done on Session.cpp regarding the use of the > toString() methods: > > In Session::sendRaw has the following code: > > 432 if ( isLoggedOn() ) result = > 433 send( message.toString(messageString) ); > > ... > ... > > 441 m_state.set( msgSeqNum, messageString > ); > > In which messageString is being changed by the > toString method as a side > effect. The problem is that it is only changed if > the session is logged > on and is being used in line 441 to store the > message in the > messageStore. > > The effect of this is that if some message is > generated while the other > party is disconnected, the message doesn't get > stored and consequently > it will not be resent upon reconnection of the other > party. > > I suggest the following change: > Lines 432, 433 should be: > > 432 messageString = message.toString(); > 433 if ( isLoggedOn() ) result = > 434 send( messageString ); > > Best regards, and once again thank you for this > excellent piece of code > > Vitor Castro > > > > ------------------------------------------------------- > This SF.Net email is sponsored by: INetU > Attention Web Developers & Consultants: Become An > INetU Hosting Partner. > Refer Dedicated Servers. We Manage Them. You Get 10% > Monthly Commission! > INetU Dedicated Managed Hosting > http://www.inetu.net/partner/index.php > _______________________________________________ > Quickfix-developers mailing list > Qui...@li... > https://lists.sourceforge.net/lists/listinfo/quickfix-developers __________________________________ Do you Yahoo!? SBC Yahoo! DSL - Now only $29.95 per month! http://sbc.yahoo.com |
From: Vitor C. <vc...@hi...> - 2003-06-18 19:29:28
|
Yes that is of course much more intelligent. How can that be checked in? Will you do it? -----Original Message----- From: qui...@li... [mailto:qui...@li...] On Behalf Of Oren Miller Sent: quarta-feira, 18 de Junho de 2003 20:20 To: Vitor Castro; 'Miller, Oren' Cc: qui...@li... Subject: Re: [Quickfix-developers] BUG in Session.cpp. Messages not being resent. What do you think about instead pulling: messageString = message.toString(); up to line 413, and passing the messageString into send on lines 425 and 433. --- Vitor Castro <vc...@hi...> wrote: > Hi Oren, > > I found what I think is a bug introduced during the > last optimization > done on Session.cpp regarding the use of the > toString() methods: > > In Session::sendRaw has the following code: > > 432 if ( isLoggedOn() ) result = > 433 send( message.toString(messageString) ); > > ... > ... > > 441 m_state.set( msgSeqNum, messageString > ); > > In which messageString is being changed by the > toString method as a side > effect. The problem is that it is only changed if > the session is logged > on and is being used in line 441 to store the > message in the > messageStore. > > The effect of this is that if some message is > generated while the other > party is disconnected, the message doesn't get > stored and consequently > it will not be resent upon reconnection of the other > party. > > I suggest the following change: > Lines 432, 433 should be: > > 432 messageString = message.toString(); > 433 if ( isLoggedOn() ) result = > 434 send( messageString ); > > Best regards, and once again thank you for this > excellent piece of code > > Vitor Castro > > > > ------------------------------------------------------- > This SF.Net email is sponsored by: INetU > Attention Web Developers & Consultants: Become An > INetU Hosting Partner. > Refer Dedicated Servers. We Manage Them. You Get 10% > Monthly Commission! > INetU Dedicated Managed Hosting http://www.inetu.net/partner/index.php > _______________________________________________ > Quickfix-developers mailing list > Qui...@li... > https://lists.sourceforge.net/lists/listinfo/quickfix-developers __________________________________ Do you Yahoo!? SBC Yahoo! DSL - Now only $29.95 per month! http://sbc.yahoo.com ------------------------------------------------------- This SF.Net email is sponsored by: INetU Attention Web Developers & Consultants: Become An INetU Hosting Partner. Refer Dedicated Servers. We Manage Them. You Get 10% Monthly Commission! INetU Dedicated Managed Hosting http://www.inetu.net/partner/index.php _______________________________________________ Quickfix-developers mailing list Qui...@li... https://lists.sourceforge.net/lists/listinfo/quickfix-developers |
From: Oren M. <ore...@ya...> - 2003-06-19 13:49:57
|
I thought so too at first, but then I realized that toString needs to be called after toAdmin or toApp in case the user modified the message in their callback. So I checked in something similar to what you originally suggested. --- Vitor Castro <vc...@hi...> wrote: > Yes that is of course much more intelligent. How can > that be checked in? > Will you do it? > > > > -----Original Message----- > From: > qui...@li... > [mailto:qui...@li...] > On Behalf Of > Oren Miller > Sent: quarta-feira, 18 de Junho de 2003 20:20 > To: Vitor Castro; 'Miller, Oren' > Cc: qui...@li... > Subject: Re: [Quickfix-developers] BUG in > Session.cpp. Messages not > being resent. > > > What do you think about instead pulling: > > messageString = message.toString(); > > up to line 413, and passing the messageString into > send on lines 425 and 433. > > --- Vitor Castro <vc...@hi...> wrote: > > Hi Oren, > > > > I found what I think is a bug introduced during > the > > last optimization > > done on Session.cpp regarding the use of the > > toString() methods: > > > > In Session::sendRaw has the following code: > > > > 432 if ( isLoggedOn() ) result = > > 433 send( message.toString(messageString) ); > > > > ... > > ... > > > > 441 m_state.set( msgSeqNum, messageString > > ); > > > > In which messageString is being changed by the > > toString method as a side > > effect. The problem is that it is only changed if > > the session is logged > > on and is being used in line 441 to store the > > message in the > > messageStore. > > > > The effect of this is that if some message is > > generated while the other > > party is disconnected, the message doesn't get > > stored and consequently > > it will not be resent upon reconnection of the > other > > party. > > > > I suggest the following change: > > Lines 432, 433 should be: > > > > 432 messageString = message.toString(); > > 433 if ( isLoggedOn() ) result = > > 434 send( messageString ); > > > > Best regards, and once again thank you for this > > excellent piece of code > > > > Vitor Castro > > > > > > > > > ------------------------------------------------------- > > This SF.Net email is sponsored by: INetU > > Attention Web Developers & Consultants: Become An > > INetU Hosting Partner. > > Refer Dedicated Servers. We Manage Them. You Get > 10% > > Monthly Commission! > > INetU Dedicated Managed Hosting > http://www.inetu.net/partner/index.php > > _______________________________________________ > > Quickfix-developers mailing list > > Qui...@li... > > > https://lists.sourceforge.net/lists/listinfo/quickfix-developers > > __________________________________ > Do you Yahoo!? > SBC Yahoo! DSL - Now only $29.95 per month! > http://sbc.yahoo.com > > > ------------------------------------------------------- > This SF.Net email is sponsored by: INetU > Attention Web Developers & Consultants: Become An > INetU Hosting Partner. > Refer Dedicated Servers. We Manage Them. You Get 10% > Monthly Commission! > INetU Dedicated Managed Hosting > http://www.inetu.net/partner/index.php > _______________________________________________ > Quickfix-developers mailing list > Qui...@li... > https://lists.sourceforge.net/lists/listinfo/quickfix-developers > __________________________________ Do you Yahoo!? SBC Yahoo! DSL - Now only $29.95 per month! http://sbc.yahoo.com |
From: Vitor C. <vc...@hi...> - 2003-06-19 15:22:33
|
Great! You're right. Thank you. -----Original Message----- From: qui...@li... [mailto:qui...@li...] On Behalf Of Oren Miller Sent: quinta-feira, 19 de Junho de 2003 14:50 To: Vitor Castro; 'Miller, Oren' Cc: qui...@li... Subject: RE: [Quickfix-developers] BUG in Session.cpp. Messages not being resent. I thought so too at first, but then I realized that toString needs to be called after toAdmin or toApp in case the user modified the message in their callback. So I checked in something similar to what you originally suggested. --- Vitor Castro <vc...@hi...> wrote: > Yes that is of course much more intelligent. How can > that be checked in? > Will you do it? > > > > -----Original Message----- > From: > qui...@li... > [mailto:qui...@li...] > On Behalf Of > Oren Miller > Sent: quarta-feira, 18 de Junho de 2003 20:20 > To: Vitor Castro; 'Miller, Oren' > Cc: qui...@li... > Subject: Re: [Quickfix-developers] BUG in > Session.cpp. Messages not > being resent. > > > What do you think about instead pulling: > > messageString = message.toString(); > > up to line 413, and passing the messageString into > send on lines 425 and 433. > > --- Vitor Castro <vc...@hi...> wrote: > > Hi Oren, > > > > I found what I think is a bug introduced during > the > > last optimization > > done on Session.cpp regarding the use of the > > toString() methods: > > > > In Session::sendRaw has the following code: > > > > 432 if ( isLoggedOn() ) result = > > 433 send( message.toString(messageString) ); > > > > ... > > ... > > > > 441 m_state.set( msgSeqNum, messageString > > ); > > > > In which messageString is being changed by the > > toString method as a side > > effect. The problem is that it is only changed if > > the session is logged > > on and is being used in line 441 to store the > > message in the > > messageStore. > > > > The effect of this is that if some message is > > generated while the other > > party is disconnected, the message doesn't get > > stored and consequently > > it will not be resent upon reconnection of the > other > > party. > > > > I suggest the following change: > > Lines 432, 433 should be: > > > > 432 messageString = message.toString(); > > 433 if ( isLoggedOn() ) result = > > 434 send( messageString ); > > > > Best regards, and once again thank you for this > > excellent piece of code > > > > Vitor Castro > > > > > > > > > ------------------------------------------------------- > > This SF.Net email is sponsored by: INetU > > Attention Web Developers & Consultants: Become An > > INetU Hosting Partner. > > Refer Dedicated Servers. We Manage Them. You Get > 10% > > Monthly Commission! > > INetU Dedicated Managed Hosting > http://www.inetu.net/partner/index.php > > _______________________________________________ > > Quickfix-developers mailing list > > Qui...@li... > > > https://lists.sourceforge.net/lists/listinfo/quickfix-developers > > __________________________________ > Do you Yahoo!? > SBC Yahoo! DSL - Now only $29.95 per month! http://sbc.yahoo.com > > > ------------------------------------------------------- > This SF.Net email is sponsored by: INetU > Attention Web Developers & Consultants: Become An > INetU Hosting Partner. > Refer Dedicated Servers. We Manage Them. You Get 10% > Monthly Commission! > INetU Dedicated Managed Hosting http://www.inetu.net/partner/index.php > _______________________________________________ > Quickfix-developers mailing list > Qui...@li... > https://lists.sourceforge.net/lists/listinfo/quickfix-developers > __________________________________ Do you Yahoo!? SBC Yahoo! DSL - Now only $29.95 per month! http://sbc.yahoo.com ------------------------------------------------------- This SF.Net email is sponsored by: INetU Attention Web Developers & Consultants: Become An INetU Hosting Partner. Refer Dedicated Servers. We Manage Them. You Get 10% Monthly Commission! INetU Dedicated Managed Hosting http://www.inetu.net/partner/index.php _______________________________________________ Quickfix-developers mailing list Qui...@li... https://lists.sourceforge.net/lists/listinfo/quickfix-developers |