slavas490 / v8-juice

code.google.com/p/v8-juice
Other
0 stars 0 forks source link

cvv8::V8Shell exception handling kaputt #29

Open GoogleCodeExporter opened 8 years ago

GoogleCodeExporter commented 8 years ago
There are a couple inconsistencies in how JS exceptions are handles in the 
V8Shell code and some routines (e.g. ExecuteFile()) are trying to report the 
problem but v8 isn't making it visible to me after they throw. i can't get 
v8::TryCatch to do a damned thing reliably, which makes its current usage in 
the API useless. The reason the functions take a TryCatch object is historical: 
i could not get exceptions thrown from load()ed code back to the client 
properly if i used an internal TryCatch.

Anyway... sit down one day and clean up the interface. See if we can use 
TryCatch::ReThrow and remove the TryCatch parameters from the interface.

DOCUMENT YOUR CODE, V8 DEVS!!!

Original issue reported on code.google.com by sgbeal@googlemail.com on 21 Jul 2011 at 2:16

GoogleCodeExporter commented 8 years ago
See: 
http://groups.google.com/group/v8-users/browse_thread/thread/6567fb364ce7b3a5

this is directly related to issue #24 - they derive from the same code.

Original comment by sgbeal@googlemail.com on 21 Jul 2011 at 8:40