You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@harmony.apache.org by "Gregory Shimansky (JIRA)" <ji...@apache.org> on 2008/03/28 15:00:24 UTC

[jira] Resolved: (HARMONY-5656) [drlvm][port][signals] VM does not show assertion dialog on failed asserts

     [ https://issues.apache.org/jira/browse/HARMONY-5656?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Gregory Shimansky resolved HARMONY-5656.
----------------------------------------

       Resolution: Fixed
    Fix Version/s: 5.0M6

Patch is applied at 642236.

> [drlvm][port][signals] VM does not show assertion dialog on failed asserts
> --------------------------------------------------------------------------
>
>                 Key: HARMONY-5656
>                 URL: https://issues.apache.org/jira/browse/HARMONY-5656
>             Project: Harmony
>          Issue Type: Bug
>          Components: DRLVM
>         Environment: Windows
>            Reporter: Ilya Berezhniuk
>            Assignee: Gregory Shimansky
>             Fix For: 5.0M6
>
>         Attachments: assert_patch_03_2.patch
>
>
> When an assertion fired, VM does not show assertion dialog on Windows regardless of "vm.assert_dialog" property.
> The only case when assertion is displayed in a dialog is when crash handler is not initialized yet.
> The problem appears because in current crash handler design all CRT events are set up in crash handler initialization to print comsole message and do not display a dialog.
> The sceme supposed to re-enable CRT event setting in VEH. The problem with assert() is that the signal is caught neither in VEH nor in CRT handler.
> I can see 2 approaches to fix the problem:
> 1) Change CRT settings immediately when crash handler output flags are changed. This method introduces additional platform dependency and thus does not look good for me, but it shoul work stably.
> 2) SIGABRT handler can be registered on Windows, but Unix-like signals handling looks ugly in Windows environment.
> I'm going to try both methods, and provide a patch for working one.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.