msama / jinjector

Automatically exported from code.google.com/p/jinjector
0 stars 0 forks source link

Code review request #10

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
Purpose of code changes on this branch:

Make a parameter configurable for BlackBerry JInjector usage.  BlackBerry 
applications always have classes that extend UiApplication as the "main" 
class of the program.  Additionally, it is possible to exit an application 
through other means than just shutting down the event thread (System.exit, 
throw an uncaught exception on the event thread, etc...) and so we need to 
be able to tell JInjector when to log the coverage information.  
Previously, the location was hard coded.  Now, the class can be specified.

The contract for the class is that it must have a void run() method.  
Implementing Runnable makes sense but is not necessary.  When the run() 
method *begins* executing, the coverage information will be logged, thus 
something like an EndOfTestListener/Runnable would be the ideal target 
here.

When reviewing my code changes, please focus on:

After the review, I'll merge this branch into:
/trunk

Original issue reported on code.google.com by sd.woodw...@gmail.com on 17 May 2010 at 4:44