This is now fixed in master as well.

On 16/9/2010 3:24 PM, Ben van Klinken wrote:
i can confirm that this was a problem. 0112a72 fixes it
ben

2010/9/16 Šplíchal Jiří <splic...@tovek.cz <mailto:splic...@tovek.cz>>

    It may not be equivalent with ASSERT macro, but on my system, the
    condition isn’t evaluated. I tried it.

    And if you look at http://en.wikipedia.org/wiki/Assert.h, is tells
    the same. If  NDEBUG is defined, than

    assert it simply ((void)0). May be it behaves differently on other
    platforms.

    Jiri

    *From:* Itamar Syn-Hershko [mailto:ita...@code972.com
    <mailto:ita...@code972.com>]
    *Sent:* Wednesday, September 15, 2010 9:02 PM
    *To:* clucene-developers@lists.sourceforge.net
    <mailto:clucene-developers@lists.sourceforge.net>
    *Subject:* Re: [CLucene-dev] Thread handling

    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  
<mailto: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
    <mailto: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
------------------------------------------------------------------------------
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