public inbox for frysk-bugzilla@sourceware.org
help / color / mirror / Atom feed
* [Bug general/4299] New: System.exit() usage
@ 2007-03-30 15:25 mark at klomp dot org
  2007-06-20  8:38 ` [Bug general/4299] " mark at klomp dot org
  0 siblings, 1 reply; 2+ messages in thread
From: mark at klomp dot org @ 2007-03-30 15:25 UTC (permalink / raw)
  To: frysk-bugzilla

As pointed out in a comment on bug #4174 System.exit() should not be used to
bail out in bad situations because it doesn't give the gui or the fhpd (or
anything else using frysk as library) a chance to properly notify the user what
happened (and clean things up).

The following classes use System.exit() at this time:

- frysk.rt.RunState
- frysk.util.FCatch
- frysk.util.StacktraceAction
- frysk.util.CoredumpAction
- frysk.gui.srcwin.SourceWindowFactory
- frysk.gui.common.Messages
- frysk.gui.Gui (this is probably the only class that should use it)

-- 
           Summary: System.exit() usage
           Product: frysk
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: general
        AssignedTo: frysk-bugzilla at sourceware dot org
        ReportedBy: mark at klomp dot org


http://sourceware.org/bugzilla/show_bug.cgi?id=4299

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


^ permalink raw reply	[flat|nested] 2+ messages in thread

* [Bug general/4299] System.exit() usage
  2007-03-30 15:25 [Bug general/4299] New: System.exit() usage mark at klomp dot org
@ 2007-06-20  8:38 ` mark at klomp dot org
  0 siblings, 0 replies; 2+ messages in thread
From: mark at klomp dot org @ 2007-06-20  8:38 UTC (permalink / raw)
  To: frysk-bugzilla


------- Additional Comments From mark at klomp dot org  2007-06-20 08:38 -------
The frysk.util.CoredumpAction one was fixed by:

        2007-06-19  Phil Muldoon  <pmuldoon@redhat.com>
        
        * CoredumpAction (AbandonCoreDump): Do not use system.exit,
        instead throw an exception.


-- 


http://sourceware.org/bugzilla/show_bug.cgi?id=4299

------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2007-06-20  8:38 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2007-03-30 15:25 [Bug general/4299] New: System.exit() usage mark at klomp dot org
2007-06-20  8:38 ` [Bug general/4299] " mark at klomp dot org

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).