If assert really isn't equivalent to the ASSERT macro, where the statement is being called but the condition isn't evaluated, this should probably be changed to reflect this (or moved to such a macro to spare the extra var).

On 15/9/2010 9:36 AM, Šplíchal Jiří wrote:

Hi,

I found following code in threads.cpp:

_LUCENE_THREADID_TYPE mutex_thread::CreateThread(luceneThreadStartRoutine* func, void* arg){

          _LUCENE_THREADID_TYPE ret;

          assert(pthread_create(&ret, NULL, func, arg) == 0 );

          return ret;

      }

This code is used if you have pthread library (I don't) but for release this cannot work because the content of the assert is not executed.

Does someone use pthreads and can test it?

--

Jiří Šplíchal

TOVEK, spol. s r.o.

splic...@tovek.cz <mailto:splic...@tovek.cz>

+420 606671930


------------------------------------------------------------------------------
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev


_______________________________________________
CLucene-developers mailing list
CLucene-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/clucene-developers
------------------------------------------------------------------------------
Start uncovering the many advantages of virtual appliances
and start using them to simplify application deployment and
accelerate your shift to cloud computing.
http://p.sf.net/sfu/novell-sfdev2dev
_______________________________________________
CLucene-developers mailing list
CLucene-developers@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/clucene-developers

Reply via email to