| Type | Bug | Status | dismissed | Date | 29-Jun-2009 13:59 |
|---|---|---|---|---|---|
| Version | alpha 64 | Category | n/a | Submitted by | meijeru |
| Platform | All | Severity | not a bug | Priority | normal |
| Summary | LOAD refuses molded errors with codes < 100 |
|---|---|
| Description |
see code; this is in addition to report #1003 presumably there is a reason for not molding them with [code: integer] because that would solve the read back problem |
| Example code |
>> mold throw 1 == "make error! 2" ; Invalid syntax >> mold make error! [code: 2] == "make error! 2" ; Invalid syntax >> make error! 2 ** Script error: invalid argument: 2 ; This is correct >> mold/all make error! [code: 2] == "#[error! 2]" ; assuming the repair will have been made >> #[error! 2] ** Syntax error: invalid constructor: [error! 2] ; can't read back |
| Assigned to | n/a | Fixed in | alpha 65 | Last Update | 30-Jun-2009 02:13 |
|---|
| Comments | |
|---|---|
|
(0001120)
BrianH 29-Jun-2009 18:08 |
We need to consider whether we should allow making errors with codes < 100 explicitly. It seems to me that the real bug is:
>> make error! [code: 2] ** Throw error: no catch for throw: make error! 2 Perhaps this should throw a script error instead. The only advantage to allowing the creation of errors < 100 is for advanced code that wants to recreate RETURN, EXIT, THROW, QUIT and HALT when those functions have been locally overridden. Since those overrides would most likely be done for sandboxing security, the ability create those errors with MAKE error! or #[error! ...] seems like a security hole to me. |
|
(0001128)
Carl 29-Jun-2009 19:18 |
It seems reasonable to make that restriction. We need a good error message for it. |
|
(0001132)
Carl 29-Jun-2009 19:49 |
"This condition is now a feature." See Brian's note. |
| Date | User | Field | Action | Change |
|---|---|---|---|---|
| 30-Jun-2009 02:13 | BrianH | Fixedin | Modified | => alpha 65 |
| 29-Jun-2009 19:49 | carl | Comment : 0001132 | Added | - |
| 29-Jun-2009 19:48 | carl | Severity | Modified | minor => not a bug |
| 29-Jun-2009 19:48 | carl | Status | Modified | reviewed => dismissed |
| 29-Jun-2009 19:28 | carl | Comment : 0001128 | Modified | - |
| 29-Jun-2009 19:27 | carl | Comment : 0001128 | Modified | - |
| 29-Jun-2009 19:18 | carl | Comment : 0001128 | Added | - |
| 29-Jun-2009 18:09 | BrianH | Comment : 0001120 | Modified | - |
| 29-Jun-2009 18:08 | BrianH | Comment : 0001120 | Added | - |
| 29-Jun-2009 18:00 | BrianH | Description | Modified | - |
| 29-Jun-2009 18:00 | BrianH | Code | Modified | - |
| 29-Jun-2009 18:00 | BrianH | Status | Modified | submitted => reviewed |
| 29-Jun-2009 13:59 | meijeru | Ticket | Added | - |