Compile error with latest quickfast from repo

[GoogleCodeExporter]

I had 3 issues building the repo head of quickfast with boost 1.51 and gcc 
4.7.2 on 64bit linux.  Below are the details and fixes in case it helps someone.

Jason

Issue 1:

    Error:

../../src/Messages/FieldIdentity_fwd.h:18:13: error: ‘intrusive_ptr’ in 
namespace ‘boost’ does not name a type
../../src/Messages/FieldIdentity_fwd.h:20:13: error: ‘intrusive_ptr’ in 
namespace ‘boost’ does not name a type

    Fix:

Add the following to src/Messages/FieldIdentity_fwd.h:
#include <boost/intrusive_ptr.hpp>

Issue 2:

    Error: Tests/testErrorRecovery.cpp:214:1: error: cast from ‘void*’ to ‘QuickFAST::int32 {aka int}’ loses precision [-fpermissive]

(many more lines with the same error)

    Fix:

Replace 'reinterpret_cast<int32>' with 'reinterpret_cast<intptr_t>'

Issue 3:

    Error: testErrorRecovery.cpp:(.text._ZNSt20__copy_move_backwardILb0ELb0ESt26random_access_iterator_tagE13__copy_move_bIPN5boost13intrusive_ptrIKN9QuickFAST8Messages13FieldIdentityEEESA_EET0_T_SC_SB_[_ZNSt20__copy_move_backwardILb0ELb0ESt26random_access_iterator_tagE13__copy_move_bIPN5boost13intrusive_ptrIKN9QuickFAST8Messages13FieldIdentityEEESA_EET0_T_SC_SB_]+0x42): undefined reference to `QuickFAST::Messages::intrusive_ptr_add_ref(QuickFAST::Messages::FieldIdentity const*)'

    Fix: 

Added '#include <Messages/FieldIdentity.h>' to src/Tests/testErrorRecovery.cpp

Original issue reported on code.google.com by ja...@alphaworkscapital.com on 5 Nov 2012 at 5:53

[GoogleCodeExporter]

My message above refers to issues which were automatically linked to quickfast 
issues in this forum.  The links were not intended.

Original comment by ja...@alphaworkscapital.com on 5 Nov 2012 at 5:54

[GoogleCodeExporter]

Issue #1: #include <boost/intrusive_ptr.hpp> is in QuickFASTPch.h which should 
be included by any build of QuickFAST code (see .../Application/QuickFAST.h)

#Issue #2: Accepted and will be fixed.  Thanks.

#Issue #3: Accepted and will be fixed.  Thanks again.

Original comment by dale.wil...@gmail.com on 21 Jan 2013 at 4:40

• Changed state: Accepted