Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consideration around error handling #2

Open
7k8m opened this issue Feb 21, 2016 · 2 comments
Open

Consideration around error handling #2

7k8m opened this issue Feb 21, 2016 · 2 comments

Comments

@7k8m
Copy link
Owner

7k8m commented Feb 21, 2016

No description provided.

@7k8m
Copy link
Owner Author

7k8m commented Feb 21, 2016

I found error handling in javascript is difficult, because for complex behavior around execution context like as "this" and amount of asynchronous processing. :(

I tried some hacking implementation in following commits ...

commit abcc127
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Sun Feb 21 22:29:24 2016 +0900

add some comment for hack and test is changed.

commit 91dc233
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Sun Feb 21 21:30:56 2016 +0900

make it possible for user to touch defaultEmitter

commit c705e2d
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Sun Feb 21 21:21:57 2016 +0900

hack to catch errors of user process to some extent.

commit 1a21e89
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Sun Feb 21 21:10:45 2016 +0900

Not happy default emitter for errors hard to trace. :(

commit 23372a9
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Sun Feb 21 20:03:07 2016 +0900

Questioable implementation of handling error event

commit cfb4d82
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Sun Feb 21 17:52:47 2016 +0900

remove mistakenly added event handler to exec

commit cebf354
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Sun Feb 21 17:49:47 2016 +0900

Listener for error is firstly coded.

@7k8m
Copy link
Owner Author

7k8m commented Feb 22, 2016

More commits related to error handling. 😑

commit a8b846e
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Mon Feb 22 22:17:46 2016 +0900

add some test case for testError.js

commit cfc7715
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Mon Feb 22 21:46:40 2016 +0900

Add some description around error handling

commit 9f4bedf
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Mon Feb 22 21:26:46 2016 +0900

Implement filed executer error listner

commit d0699b5
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Mon Feb 22 20:56:28 2016 +0900

Widen where error listner of each executer covers

commit 62ea1c1
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Mon Feb 22 20:46:58 2016 +0900

Enhance testError.js and fix bug around error processing

commit 0540c23
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Mon Feb 22 06:30:49 2016 +0900

Pass event emitter to useProcess

commit 6be1b50
Author: Tomohito Nakayama tmnk@nysalor.local
Date: Sun Feb 21 23:19:40 2016 +0900

Hack error processing where possible.

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

No branches or pull requests

1 participant