Not signed in (Sign In)

Vanilla 1.1.4 is a product of Lussumo. More Information: Documentation, Community Support.

    • CommentAuthorphlion
    • CommentTimeSep 24th 2012 edited
     

    Hi Matthias,

    We are trying the functions of 0.22, and encountered a problem.

    The procedure is to start klayout and open a db, click Edit => Select => Disable All

    In 0.21.19 it unchecks all options, and in 0.22 it response the error:

    *** ERROR: /klayout-0.22/src/layAbstractMenu.cc,359,mp_handle
    *** ERROR: Internal error: /klayout-0.22/src/layAbstractMenu.cc:359 mp_handle was not true
    

    Best Regards,

    chhung

    • CommentAuthorMatthias
    • CommentTimeSep 25th 2012 edited
     

    Hi chhung,

    These methods are supposed to replace the script we created some time ago. Apparently the built-in method is not working. That's embarrasing. I'll try to fix it and provide a patch.

    Thank you for that report,

    Matthias

    • CommentAuthorMatthias
    • CommentTimeSep 25th 2012 edited
     

    Hi chhung,

    here is a patch for that problem: replace the following lines at layAbstractMenu.cc, 357++:

    tl_assert (mp_handle);
    tl_assert (mp_handle->ptr ());
    return mp_handle->ptr ();
    

    with

    return mp_handle ? mp_handle->ptr () : 0;
    

    That basically disables the assertion but in that case the assertion was not useful. I'll include that fix in the next maintenance release.

    Best regards,

    Matthias

    • CommentAuthorphlion
    • CommentTimeSep 27th 2012
     

    Hi Matthias,

    This patch fixed the problem, thanks :)

    Best Regards,

    chhung