[Quickfix-developers] QickFix 1.3.1 : thread_spawn

[Manuel H. <mhe...@ca...>]

Hi all,

I am using QuickFix 1.3.1.
I found the following probleme:

There are 2 methods allowing to spawn threads in the file Utility.h
    bool thread_spawn( void*( *func ) ( void* ), void* var, int& thread )
    bool thread_spawn( void*( *func ) ( void* ), void* var )

he second one is used files Initiator.cpp and Acceptor.cpp (at line Accep=
tor.cpp:185 and Initiator.cpp:198):
    int threadid =3D thread_spawn( &startThread, this );

So in this line, the variable 'threadid' receive a boolean and not a thre=
ad id.

What I suggest in file Utility.h ate line 103 is :
replace
                 int thread_spawn( void*( *func ) ( void* ), void* var );
by
                 bool thread_spawn( void*( *func ) ( void* ), void* var )=
;


and replace in file Utility.cpp at line 251
      bool thread_spawn( void*( *func ) ( void* ), void* var )
      {
         int thread =3D 0;
         return thread_spawn( func, var, thread );
      }
by
      int thread_spawn( void*( *func ) ( void* ), void* var )
      {
         int thread =3D 0;
         thread_spawn( func, var, thread );
         return thread;
      }

I don't know who is involved in checking these modifications and putting =
them in the official code.
Can somebody help me ?

Thank you.

Manuel HENRIQUES

"This message,  including any attachments may contain confidential and
privileged material;  it is intended only for the person to whom it is
addressed.  Its  contents  do not  constitute  a commitment  by Cr=E9dit
Agricole Indosuez Cheuvreux  except  where  provided  for in a written
agreement.  Cr=E9dit Agricole Indosuez Cheuvreux assumes no liability or
responsibility for the consequences arising out of a delay and/or loss
in  transit  of  this  message,  or for corruption  or other  error(s)
arising in its transmission and for any misuse or fraudulent use which
may  be made thereof.  If you are not the  intended recipient,  please
contact us  and abstain from  any disclosure, use or dissemination. To
the extent  that  this message  contains  research information  and/or
recommendations,  these  are  provided  on  the  same  basis as Cr=E9dit
Agricole Indosuez Cheuvreux's  published  research  and  the recipient
must  have  regard  to  all   disclosures  and  disclaimers  contained
therein."