Skip to content

Added unexpected error message to the player view#3

Merged
joethephish merged 2 commits intomasterfrom
catch-inklecate-crash
May 29, 2016
Merged

Added unexpected error message to the player view#3
joethephish merged 2 commits intomasterfrom
catch-inklecate-crash

Conversation

@DiogoNeves
Copy link
Collaborator

Just simply shows the stderr message in the editor

fixes #2

Just simply shows the stderr message in the editor
events.unexpectedExit();
});

ipc.on("play-story-unexpected-error", (event, error, fromSessionId) => {
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I decided to create a new message type but let me know if you prefer to change play-story-unexpected-exit instead

@joethephish joethephish merged commit 0c51478 into master May 29, 2016
@DiogoNeves DiogoNeves deleted the catch-inklecate-crash branch May 29, 2016 17:07
bjennings76 added a commit to bjennings76/inky that referenced this pull request Feb 1, 2018
- Added a generic 'escapes' section that should capture most escape locations
- Separated out 'gather' into a push/pop group
- Added escapes, comments, and a 'logicLineInsert' type into push area of 'gather'
- Added support for comments after VARs
- Removed custom markup for tag escapes since it's now build-in at a higher level
- Added push section to tags to support post-tag comments
- Added escapes to all statements in general
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

When inklecate crashes, we should handle it specially

2 participants