public inbox for insight@sourceware.org
 help / color / mirror / Atom feed
From: "Dave Korn" <dave.korn@artimi.com>
To: "'Andrew STUBBS'" <andrew.stubbs@st.com>, <insight@sources.redhat.com>
Subject: RE: MinGW setjmp SEGV
Date: Fri, 05 Aug 2005 15:28:00 -0000	[thread overview]
Message-ID: <SERRANOu4A9FKQ02gKW000000ce@SERRANO.CAM.ARTIMI.COM> (raw)
In-Reply-To: <op.su1hpkl6o669wz@terrorhawk.bri.st.com>

----Original Message----
>From: Andrew STUBBS
>Sent: 05 August 2005 15:40


> Many apologies. I have got the diff backwards! It is tcl 8.4.1 that is
> broken and AFAIKT CVS HEAD is little altered.
> 
> Please find a correct one attached.
> 
> Andrew Stubbs

  Aha, that makes more sense, thanks.

  I don't understand the need for it yet, though; can you enlarge on the
problem or point me at a tcl.sourceforge bugzilla report?  This patch would
affect cygwin as well as mingw, since it also defines HAVE_NO_SEH; AFAIUI,
the macro means 'have no seh support in the compiler' rather than 'have no
seh in the target OS', but cygwin (apparently) is happy enough using 'doze
SEH, and I don't understand the comment about "Unlike Borland and Microsoft
we don't ....  pushing registration records onto the runtime stack."  Yet
the EXCEPTION_REGISTRATION variable "registration" is on the stack and
appears to be linked into the 'doze SEH chain in pretty much the usual
fashion.  I'm sure this is just my lack of comprehension.

  OTOH this patch would seem to address my concerns about the reentrancy
problems of using static _ESP and _EBP variables.

  Should or shouldn't the same changes be made to the exception handling in
tclWinFCmd.c and tclWinChan.c as well?

    cheers,
      DaveK
-- 
Can't think of a witty .sigline today....

  reply	other threads:[~2005-08-05 15:28 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2005-08-05 13:44 Andrew STUBBS
2005-08-05 14:01 ` Dave Korn
2005-08-05 14:40   ` Andrew STUBBS
2005-08-05 15:28     ` Dave Korn [this message]
2005-08-05 15:57       ` Andrew STUBBS
2005-08-05 16:35         ` Dave Korn
2005-08-05 21:43 ` Steven Johnson
2005-08-08  9:38   ` Andrew STUBBS
2005-09-21  2:25 ` Dave Murphy
2005-09-21  9:00   ` Andrew STUBBS
2005-09-21 21:25     ` Dave Murphy

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=SERRANOu4A9FKQ02gKW000000ce@SERRANO.CAM.ARTIMI.COM \
    --to=dave.korn@artimi.com \
    --cc=andrew.stubbs@st.com \
    --cc=insight@sources.redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).