Skip to content

Commit e8a2ccf

Browse files
authored
Merge pull request #2 from kenhys/fix-removed-mesurements
Remove unavailable measure mode
2 parents b6fa2a9 + 95df170 commit e8a2ccf

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

lib/fluent/rubyprof.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -69,7 +69,7 @@ def parse_options(argv = ARGV)
6969
raise OptionParser::InvalidOption.new("`start` or `stop` must be specified as the 1st argument")
7070
end
7171

72-
measure_modes = %w[PROCESS_TIME WALL_TIME CPU_TIME ALLOCATIONS MEMORY GC_RUNS GC_TIME]
72+
measure_modes = %w[PROCESS_TIME WALL_TIME ALLOCATIONS MEMORY]
7373
unless measure_modes.include?(opts[:measure_mode])
7474
raise OptionParser::InvalidOption.new("-m allows one of #{measure_modes.join(', ')}")
7575
end

spec/fluent-rubyprof/rubyprof_spec.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
end
1414

1515
it 'correct measure_mode' do
16-
expect { Fluent::Rubyprof.new.parse_options(['start', '-m', 'CPU_TIME']) }.not_to raise_error
16+
expect { Fluent::Rubyprof.new.parse_options(['start', '-m', 'PROCESS_TIME']) }.not_to raise_error
1717
end
1818

1919
it 'incorrect measure_mode' do

0 commit comments

Comments
 (0)