Skip to content

Removed functions.util.* as they don't seem to be used (for many years).#335

Closed
ham1 wants to merge 1 commit intoapache:trunkfrom
ham1:remove_unused_functions.util
Closed

Removed functions.util.* as they don't seem to be used (for many years).#335
ham1 wants to merge 1 commit intoapache:trunkfrom
ham1:remove_unused_functions.util

Conversation

@ham1
Copy link
Contributor

@ham1 ham1 commented Nov 21, 2017

Description

Remove files which don't seem to be used, they have a TODO comment from 2008 and hasn't been meaningfully changed since 2003.

How Has This Been Tested?

Unit tests.

@codecov-io
Copy link

Codecov Report

Merging #335 into trunk will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##              trunk     #335      +/-   ##
============================================
+ Coverage     57.91%   57.91%   +<.01%     
+ Complexity    10025    10024       -1     
============================================
  Files          1148     1146       -2     
  Lines         73789    73781       -8     
  Branches       7328     7328              
============================================
- Hits          42734    42733       -1     
+ Misses        28577    28570       -7     
  Partials       2478     2478
Impacted Files Coverage Δ Complexity Δ
...c/core/org/apache/jmeter/reporters/Summariser.java 85.38% <0%> (-0.77%) 18% <0%> (-1%)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5fc60e7...38a6e30. Read the comment docs.

@asfgit asfgit closed this in 4bdb53d Nov 23, 2017
@ham1 ham1 deleted the remove_unused_functions.util branch November 23, 2017 22:15
asfgit pushed a commit that referenced this pull request Nov 25, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants