#9943: Replace "error" with "errorWithStackTrace" from GHC.Stack in base libs -------------------------------------+------------------------------------- Reporter: spacekitteh | Owner: Type: task | spacekitteh Priority: normal | Status: new Component: Core Libraries | Milestone: 7.12.1 Resolution: | Version: Operating System: Unknown/Multiple | Keywords: Type of failure: None/Unknown | Architecture: Blocked By: | Unknown/Multiple Related Tickets: | Test Case: | Blocking: | Differential Revisions: -------------------------------------+------------------------------------- Comment (by hvr): Btw, it's not as simple as just set `error = errorWithStackTrace`; there needs to be a way to recover the string passed to `error` w/o the additional stack-trace from the `ErrorCall` exception value (which currently provides the originally passed `error`-message as its payload). {{{#!hs -- |This is thrown when the user calls 'error'. The @String@ is the -- argument given to 'error'. newtype ErrorCall = ErrorCall String deriving (Eq, Ord, Typeable) instance Exception ErrorCall instance Show ErrorCall where showsPrec _ (ErrorCall err) = showString err }}} See also this short conversation for some ideas: {{{ hvr> carter: btw, was my concern regarding modifying the payload of the ErrorCall exception ever addressed? carter> nopeee! :) carter> maybe we expose a pattern synonym as the constructor name carter> and have the real constructor have an extra field for the trace? hvr> could work carter> so show will still render the full mess carter> but everyone's old code wont break carter> because unless youre explicity catching it carter> it ends up just being showed right? carter> error "fix me" carter> hvr: pattern synonyms dont need to be eneabled in the use sites right? carter> hvr: err, the only reason for that change would be backwards compat right? hvr> carter: well, actually it's a nice property to be able to recover the exact string you pass to 'error' carter> hvr: i wasn't arguing against it :) hvr> carter: regardless of backward compat carter> ok carter> so breaking change to make it have two args? carter> or hide it with patternsyn? carter> or wait a year and try again? hvr> carter: you've got one year to figure out the perfect way :) carter> fineeeee carter> solution: break everything hvr> carter: otoh, maybe by then we may have some other facility which makes this redundant }}} -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/9943#comment:2 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#9943: Replace "error" with "errorWithStackTrace" from GHC.Stack in base libs -------------------------------------+------------------------------------- Reporter: spacekitteh | Owner: Type: task | spacekitteh Priority: normal | Status: closed Component: Core Libraries | Milestone: Resolution: invalid | Version: Operating System: Unknown/Multiple | Keywords: Type of failure: None/Unknown | Architecture: Blocked By: | Unknown/Multiple Related Tickets: | Test Case: | Blocking: | Differential Revisions: -------------------------------------+------------------------------------- Comment (by spacekitteh): Closed ticket in light of http://thread.gmane.org/gmane.comp.lang.haskell.libraries/23525/ this thread, which subsumes this. -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/9943#comment:4 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler

#9943: Replace "error" with "errorWithStackTrace" from GHC.Stack in base libs -------------------------------------+------------------------------------- Reporter: spacekitteh | Owner: Type: task | Status: new Priority: normal | Milestone: Component: Core Libraries | Version: Resolution: | Keywords: Operating System: Unknown/Multiple | Architecture: Type of failure: None/Unknown | Unknown/Multiple Blocked By: | Test Case: Related Tickets: | Blocking: | Differential Revisions: -------------------------------------+------------------------------------- Comment (by carter): Chris and I spent some time working on this, its actually pretty tricky to implement even a nominally simple patch for this because you quickly run into navigating figuring out where to write the boot file for cutting a recursive module loop -- Ticket URL: http://ghc.haskell.org/trac/ghc/ticket/9943#comment:6 GHC http://www.haskell.org/ghc/ The Glasgow Haskell Compiler