Bug #5650
closed
Also:
actors/src/org/geon/LidarWorkflowExecute.java line 373
actors/src/org/geon/ModelService.java line 267
loader/src/org/geon/WorkflowExecute.java line 227
# lines : 176
I marked all three as deprecated.
Why are all three deprecated?
Shouldn't two be marked deprecated with tags that suggest using the third?
For example:
@deprecated This actor is unsupported and contains duplicate code from @see {org.geon.WorkflowExecute} in the loader module. Please use that actor instead.
Check the deprecated tags, they should say why the actor was deprecated and
there should be FIXMEs that say where the duplicated code is located.
Or, as you say, we could move this code into a separate package that would never
be compiled.
I took another look at these three classes and found that WorkflowExecute and ModelService are almost a subset of LidarWorkflowExecute. I removed the deprecated tag from LidarWorkflowExecute and updated the tags for WorkflowExecute and ModelService.
Original Bugzilla ID was 5650
Also available in: Atom
PDF