remove unused 'target' field from Job

the Job class has a 'target' field that isn't used,
and probably doesn't work correctly if it is used.

I was trying to write another pipeline stage, and got
confused by Job.target, and couldn't convince myself that
Job.target was going to behave correctly, then found out that
it isn't used by any reachable code.

this change deletes all the references to Job.target.

[MikeSamuel, 2009-09-10 21:55:48]

http://codereview.appspot.com/115089/diff/1/2
File src/com/google/caja/plugin/stages/ConsolidateCodeStage.java (right):

http://codereview.appspot.com/115089/diff/1/2#newcode20
Line 20: import com.google.caja.parser.ParseTreeNode;
Is this still used?

[felix8a, 2009-09-10 22:06:28]

On 2009/09/10 21:55:48, MikeSamuel wrote:
> http://codereview.appspot.com/115089/diff/1/2
> File src/com/google/caja/plugin/stages/ConsolidateCodeStage.java (right):
> 
> http://codereview.appspot.com/115089/diff/1/2#newcode20
> Line 20: import com.google.caja.parser.ParseTreeNode;
> Is this still used?

ah, yeah, I had my eclipse configured wrong, so didn't see the warning in the
noise of "dead code" warnings.

updated snapshot, also fixed other warnings.  I have 0 warnings now.

[MikeSamuel, 2009-09-10 22:10:02]

LGTM

[felix8a, 2009-09-24 19:44:22]

On 2009/09/10 22:10:02, MikeSamuel wrote:
> LGTM

@r3723