public inbox for frysk-bugzilla@sourceware.org
help / color / mirror / Atom feed
* [Bug general/3730] New: Remove system.exit() and foo.printStackTrace() from frysk/proc/ProcBlockAction.java
@ 2006-12-15  2:39 pmuldoon at redhat dot com
  2006-12-15 20:05 ` [Bug general/3730] " cagney at redhat dot com
  0 siblings, 1 reply; 2+ messages in thread
From: pmuldoon at redhat dot com @ 2006-12-15  2:39 UTC (permalink / raw)
  To: frysk-bugzilla

Exiting a system should only be decided by final implementors (ie fstack or
fcore) not by the Action. It would be more appropriate to throw exceptions and
let implementors deal with error conditions. Also foo.printStackTrace() assumes
a console, and may not be appropriate in an Action. Might be better, again, to
throw an exception and let the implementor extract/decide to print the stack
trace there.

-- 
           Summary: Remove system.exit() and foo.printStackTrace() from
                    frysk/proc/ProcBlockAction.java
           Product: frysk
           Version: unspecified
            Status: NEW
          Severity: normal
          Priority: P2
         Component: general
        AssignedTo: frysk-bugzilla at sourceware dot org
        ReportedBy: pmuldoon at redhat dot com


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

------- 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/3730] Remove system.exit() and foo.printStackTrace() from frysk/proc/ProcBlockAction.java
  2006-12-15  2:39 [Bug general/3730] New: Remove system.exit() and foo.printStackTrace() from frysk/proc/ProcBlockAction.java pmuldoon at redhat dot com
@ 2006-12-15 20:05 ` cagney at redhat dot com
  0 siblings, 0 replies; 2+ messages in thread
From: cagney at redhat dot com @ 2006-12-15 20:05 UTC (permalink / raw)
  To: frysk-bugzilla


------- Additional Comments From cagney at redhat dot com  2006-12-15 20:04 -------
Yes, good point, in general command line utilities should at least not print
stacktraces.


-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|frysk-bugzilla at sourceware|npremji at redhat dot com
                   |dot org                     |
             Status|NEW                         |ASSIGNED


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

------- 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:[~2006-12-15 20:05 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-12-15  2:39 [Bug general/3730] New: Remove system.exit() and foo.printStackTrace() from frysk/proc/ProcBlockAction.java pmuldoon at redhat dot com
2006-12-15 20:05 ` [Bug general/3730] " cagney at redhat dot com

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