Bug 47943 - [Patch] DisabledComponentRemover should die
[Patch] DisabledComponentRemover should die
Status: RESOLVED FIXED
Product: JMeter
Classification: Unclassified
Component: Main
2.3.4
All All
: P2 enhancement (vote)
: ---
Assigned To: JMeter issues mailing list
:
Depends on:
Blocks:
  Show dependency tree
 
Reported: 2009-10-06 02:26 UTC by benoit.wiart
Modified: 2009-11-14 15:11 UTC (History)
0 users



Attachments
Patch 1 : delete the call to DisabledComponentRemover from Start (1.11 KB, application/octet-stream)
2009-10-06 02:26 UTC, benoit.wiart
Details
Patch 2 : delete the DisabledComponentRemover file (2.83 KB, patch)
2009-10-06 02:27 UTC, benoit.wiart
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description benoit.wiart 2009-10-06 02:26:00 UTC
Created attachment 24347 [details]
Patch 1 : delete the call to DisabledComponentRemover from Start

In org.apache.jmeter.gui.action.Start#startEngine is the only method in jmeter
where DisabledComponentRemover is used and it's useless.
The disabled components have already been removed from the tree by the previous
instruction : JMeter#convertSubTree.

Patch 1 : delete the call to DisabledComponentRemover from Start
Patch 2 : delete the DisabledComponentRemover file

Benoit Wiart
Ubik Ingénierie
http://www.ubik-ingenierie.com
Comment 1 benoit.wiart 2009-10-06 02:27:02 UTC
Created attachment 24348 [details]
Patch 2 : delete the DisabledComponentRemover file
Comment 2 Sebb 2009-11-14 15:11:08 UTC
Thanks, fixed:

URL: http://svn.apache.org/viewvc?rev=836288&view=rev
Log:
Bug 47943 - DisabledComponentRemover is not used in Start class