benchmark-memory 0.1.2 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: ce1d2f5bdcc1fbcd47258b0baa40270454220de4
-  data.tar.gz: 624cd04313af120944dcbe193dcace591a980a17
+SHA256:
+  metadata.gz: 77c690ecb4b3640eb994fe9dbaddaac4dde593fb445b8e6fff50c771dec82c6f
+  data.tar.gz: 4f29c72bae6ec8a5d024c4c0d89ff7032a75c9c6a2dd738ecdde5d4dd3e162b9
 SHA512:
-  metadata.gz: 07cae3b39fdaac96b4f9601288cc5972db7bb8cd14a3a71d3c55d300c22e7a850db993efae450727b27a73fe29ab2e6a3b6a3dece852adb783bee25ced15c023
-  data.tar.gz: 33c02f69a9a8cca90bd528445447cce532cf93d188acc43fdd5abf3fac6cb4746c1788f35d035893f4fbd033ff652ec780195b4fc77b31c9e069d454e577afc7
+  metadata.gz: e31d03064d7de106086091ea7726d8ac239cee28f799f1536b96738bf1dbd676b192ca5e5a7862b6917770f6f3880c60d10d4d19f836e89be83ce0a3bb8357fa
+  data.tar.gz: '0789ccf4fa45943688b6a486e3816295216f0500b040ccc3843466582a8f42f0d9f3fcbc8dbb42fa439d293a4c4519f1b27460ca4ceb12100cfe5e234b4d82cf'

data/CHANGELOG.md

@@ -4,26 +4,34 @@ All notable changes to this project will be documented in this file. This projec
 
 [semver]: http://semver.org/spec/v2.0.0.html
 
-## [0.1.2] - 2017-01-30
+## [0.2.0](https://github.com/michaelherold/benchmark-memory/compare/v0.1.1...v0.2.0)
+
+### Added
+
+- [#11](https://github.com/michaelherold/benchmark-memory/pull/11): Drop support for Ruby < 2.4 - [@dblock](https://github.com/dblock).
+- [#23](https://github.com/michaelherold/benchmark-memory/pull/23): Allow for sorting the comparison by different criteria - [@michaelherold](https://github.com/michaelherold).
+
+### Updated
+
+- [#16](https://github.com/michaelherold/benchmark-memory/pull/16): Updated Rubocop to ~> 1 and Ruby to 2.5.0+ - [@AlexWayfer](https://github.com/AlexWayfer).
+- [#19](https://github.com/michaelherold/benchmark-memory/pull/19): Updated to `memory_profiler` 1.0 - [@AlexWayfer](https://github.com/AlexWayfer).
+
+## [0.1.2](https://github.com/michaelherold/benchmark-memory/compare/v0.1.1...v0.1.2) - 2017-01-30
 
 ### Fixed
 
 - [#4](https://github.com/michaelherold/benchmark-memory/pull/4): Fix a small bug when StringIO wasn't properly required - [@rzane](https://github.com/rzane).
 
-## [0.1.1] - 2016-06-10
+## [0.1.1](https://github.com/michaelherold/benchmark-memory/compare/v0.1.0...v0.1.1) - 2016-06-10
 
 ### Fixed
 
 - Printing comparisons for multiple entries.
 
-## [0.1.0] - 2016-05-18
+## [0.1.0](https://github.com/michaelherold/benchmark-memory/tree/v0.1.0) - 2016-05-18
 
 ### Added
 
 - Main `Benchmark.memory` method.
 - Holding results between invocations for measuring implementations on different versions of Ruby or different versions of libraries.
 - Quiet mode, with no command line output.
-
-[0.1.2]: https://github.com/michaelherold/benchmark-memory/compare/v0.1.1...v0.1.2
-[0.1.1]: https://github.com/michaelherold/benchmark-memory/compare/v0.1.0...v0.1.1
-[0.1.0]: https://github.com/michaelherold/benchmark-memory/tree/v0.1.0

data/README.md

@@ -1,12 +1,12 @@
 # benchmark-memory
 
-[![Build Status](https://travis-ci.org/michaelherold/benchmark-memory.svg)][travis]
+[![CI](https://github.com/michaelherold/benchmark-memory/workflows/CI/badge.svg)][ci]
 [![Code Climate](https://codeclimate.com/github/michaelherold/benchmark-memory/badges/gpa.svg)][codeclimate]
-[![Inline docs](http://inch-ci.org/github/michaelherold/benchmark-memory.svg?branch=master)][inch]
+[![Inline docs](http://inch-ci.org/github/michaelherold/benchmark-memory.svg?branch=main)][inch]
 
+[ci]: https://github.com/michaelherold/benchmark-memory/actions
 [codeclimate]: https://codeclimate.com/github/michaelherold/benchmark-memory
 [inch]: http://inch-ci.org/github/michaelherold/benchmark-memory
-[travis]: https://travis-ci.org/michaelherold/benchmark-memory
 
 benchmark-memory is a tool that helps you to benchmark the memory usage of different pieces of code. It leverages the power of [memory_profiler] to give you a metric of the total amount of memory allocated and retained by a block, as well as the number of objects and strings allocated and retained.
 
@@ -71,7 +71,7 @@ Comparison:
   dynamic allocation:         40 allocated - Infx more
 ```
 
-Reading this output shows that the "dynamic allocation" example allocates one string that is not retained outside the scope of the block. The "frozen string" example, however, does not allocate anything because it reuses the frozen string that was created during the method definition.
+Reading this output shows that the "dynamic allocation" example allocates one string that is not retained outside the scope of the block. The "frozen string" example, however, does not allocate anything because it reuses the frozen string that we created during the method definition.
 
 [benchmark-ips]: https://github.com/evanphx/benchmark-ips
 
@@ -95,7 +95,31 @@ Benchmark.memory do |x|
 end
 ```
 
-Calling `#compare!` on the job within the setup block of `Benchmark.memory` enables the output of the comparison section of the benchmark. Without it, this section is suppressed and you only get the raw numbers output during calculation.
+Calling `#compare!` on the job within the setup block of `Benchmark.memory` enables the output of the comparison section of the benchmark. Without it, the benchmark suppresses this section and you only get the raw numbers output during calculation.
+
+By default, this compares the reports by the amount of allocated memory. You can configure the comparison along two axes. The first axis is the metric, which is one of: `:memory`, `:objects`, or `:strings`. The second is the value, which is either `:allocated` or `:retained`.
+
+Depending on what you're trying to benchmark, different configurations make sense. For example:
+
+``` ruby
+Benchmark.memory do |bench|
+  bench.compare! memory: :allocated
+  # or, equivalently:
+  # bench.compare!
+end
+```
+
+The purpose of the default configuration is benchmarking the total amount of memory an algorithm might use. If you're trying to improve a memory-intensive task, this is the mode you want.
+
+An alternative comparison might look like:
+
+``` ruby
+Benchmark.memory do |bench|
+  bench.compare! memory: :retained
+end
+```
+
+When you're looking for a memory leak, this configuration can help you because it compares your reports by the amount of memory that the garbage collector does not collect after the benchmark.
 
 ### Hold results between invocations
 
@@ -107,7 +131,7 @@ end
 
 Often when you want to benchmark something, you compare two implementations of the same method. This is cumbersome because you have to keep two implementations side-by-side and call them in the same manner. Alternatively, you may want to compare how a method performs on two different versions of Ruby. To make both of these scenarios easier, you can enable "holding" on the benchmark.
 
-By calling `#hold!` on the benchmark, you enable the benchmark to write to the given file to store its results in a file that can be read in between invocations of your benchmark.
+By calling `#hold!` on the benchmark, you enable the benchmark to write to the given file to store its results in a file that can the benchmark reads in between invocations of your benchmark.
 
 For example, imagine that you have a library that exposes a method called `Statistics.calculate_monthly_recurring_revenue` that you want to optimize for memory usage because it keeps causing your worker server to run out of memory. You make some changes to the method and commit them to an `optimize-memory` branch in Git.
 
@@ -133,33 +157,34 @@ Note that the method calls are the same for both tests and that we have enabled
 You could then run the following (assuming you saved your benchmark as `benchmark_mrr.rb`:
 
 ```sh
-$ git checkout master
+$ git checkout main
 $ ruby benchmark_mrr.rb
 $ git checkout optimize-memory
 $ ruby benchmark_mrr.rb
 ```
 
-The first invocation of `ruby benchmark_mrr.rb` runs the benchmark in the "original" entry using your code in your `master` Git branch. The second invocation runs the benchmark in the "optimized" entry using the code in your `optimize-memory` Git branch. These two results are collated and then compared to show you the difference between the two.
+The first invocation of `ruby benchmark_mrr.rb` runs the benchmark in the "original" entry using your code in your `main` Git branch. The second invocation runs the benchmark in the "optimized" entry using the code in your `optimize-memory` Git branch. It then collates and compares the two results to show you the difference between the two.
 
 When enabling holding, the benchmark writes to the file passed into the `#hold!` method. After you run all of the entries in the benchmark, the benchmark automatically cleans up its log by deleting the file.
 
 ## Supported Ruby Versions
 
-This library aims to support and is [tested against][travis] the following Ruby versions:
+This library aims to support and is [tested against][ci] the following Ruby versions:
 
-* Ruby 2.1
-* Ruby 2.2
-* Ruby 2.3
+* Ruby 2.5
+* Ruby 2.6
+* Ruby 2.7
+* Ruby 3.0
 
 If something doesn't work on one of these versions, it's a bug.
 
-This library may inadvertently work (or seem to work) on other Ruby versions, however, support will only be provided for the versions listed above.
+This library may inadvertently work (or seem to work) on other Ruby versions, however, we will only give support for the versions listed above.
 
-If you would like this library to support another Ruby version or implementation, you may volunteer to be a maintainer. Being a maintainer entails making sure all tests run and pass on that implementation. When something breaks on your implementation, you will be responsible for providing patches in a timely fashion. If critical issues for a particular implementation exist at the time of a major release, support for that Ruby version may be dropped.
+If you would like this library to support another Ruby version or implementation, you may volunteer to be a maintainer. Being a maintainer entails making sure all tests run and pass on that implementation. When something breaks on your implementation, you will be responsible for providing patches in a timely fashion. If critical issues for a particular implementation exist at the time of a major release, we may drop support for that Ruby version.
 
 ## Versioning
 
-This library aims to adhere to [Semantic Versioning 2.0.0][semver]. Violations of this scheme should be reported as bugs. Specifically, if a minor or patch version is released that breaks backward compatibility, that version should be immediately yanked and/or a new version should be immediately released that restores compatibility. Breaking changes to the public API will only be introduced with new major versions. As a result of this policy, you can (and should) specify a dependency on this gem using the [Pessimistic Version Constraint][pessimistic] with two digits of precision. For example:
+This library aims to adhere to [Semantic Versioning 2.0.0][semver]. Report violations of this scheme as bugs. Specifically, if we release a minor or patch version that breaks backward compatibility, that version should be immediately yanked and/or a new version should be immediately released that restores compatibility. We will only introduce breaking changes to the public API with new major versions. As a result of this policy, you can (and should) specify a dependency on this gem using the [Pessimistic Version Constraint][pessimistic] with two digits of precision. For example:
 
     spec.add_dependency "benchmark-memory", "~> 0.1"
 

data/Rakefile

@@ -1,29 +1,19 @@
-require "bundler"
+# frozen_string_literal: true
+
+require 'bundler'
 Bundler.setup
 Bundler::GemHelper.install_tasks
 
-require "inch/rake"
+require 'inch/rake'
 Inch::Rake::Suggest.new(:inch)
 
-require "rspec/core/rake_task"
+require 'rspec/core/rake_task'
 RSpec::Core::RakeTask.new(:spec)
 
-require "rubocop/rake_task"
+require 'rubocop/rake_task'
 RuboCop::RakeTask.new(:rubocop)
 
-require "yard/rake/yardoc_task"
+require 'yard/rake/yardoc_task'
 YARD::Rake::YardocTask.new(:yard)
 
-task :mutant do
-  command = [
-    "bundle exec mutant",
-    "--include lib",
-    "--require interactor-contracts",
-    "--use rspec",
-    "Interactor::Contracts*",
-  ].join(" ")
-
-  system command
-end
-
-task :default => [:spec, :rubocop, :yard, :inch]
+task default: %i[spec rubocop yard inch]

data/benchmark-memory.gemspec

@@ -1,24 +1,24 @@
-# coding: utf-8
+# frozen_string_literal: true
 
-require File.expand_path("../lib/benchmark/memory/version", __FILE__)
+require File.expand_path('lib/benchmark/memory/version', __dir__)
 
 Gem::Specification.new do |spec|
-  spec.name          = "benchmark-memory"
+  spec.name          = 'benchmark-memory'
   spec.version       = Benchmark::Memory::VERSION
-  spec.authors       = ["Michael Herold"]
-  spec.email         = ["michael.j.herold@gmail.com"]
+  spec.authors       = ['Michael Herold']
+  spec.email         = ['michael.j.herold@gmail.com']
 
-  spec.summary       = "Benchmark-style memory profiling for Ruby 2.1+"
+  spec.summary       = 'Benchmark-style memory profiling'
   spec.description   = spec.summary
-  spec.homepage      = "https://github.com/michaelherold/benchmark-memory"
-  spec.license       = "MIT"
+  spec.homepage      = 'https://github.com/michaelherold/benchmark-memory'
+  spec.license       = 'MIT'
 
-  spec.files = %w(CHANGELOG.md CONTRIBUTING.md LICENSE.md README.md Rakefile)
-  spec.files += %w(benchmark-memory.gemspec)
-  spec.files += Dir["lib/**/*.rb"]
-  spec.require_paths = ["lib"]
+  spec.files = %w[CHANGELOG.md CONTRIBUTING.md LICENSE.md README.md Rakefile]
+  spec.files += %w[benchmark-memory.gemspec]
+  spec.files += Dir['lib/**/*.rb']
+  spec.require_paths = ['lib']
 
-  spec.add_dependency "memory_profiler", "~> 0.9"
+  spec.required_ruby_version = '>= 2.5.0'
 
-  spec.add_development_dependency "bundler", "~> 1.12"
+  spec.add_dependency 'memory_profiler', '~> 1'
 end

data/lib/benchmark/memory/errors.rb

@@ -1,7 +1,13 @@
+# frozen_string_literal: true
+
 module Benchmark
   module Memory
     Error = Class.new(StandardError)
 
-    ConfigurationError = Class.new(Error)
+    ConfigurationError = Class.new(Error) do
+      def message
+        'You did not give a test block to your call to `Benchmark.memory`'
+      end
+    end
   end
 end

data/lib/benchmark/memory/held_results/entry_serializer.rb

@@ -1,6 +1,8 @@
-require "benchmark/memory/held_results/serializer"
-require "benchmark/memory/held_results/measurement_serializer"
-require "benchmark/memory/report/entry"
+# frozen_string_literal: true
+
+require 'benchmark/memory/held_results/serializer'
+require 'benchmark/memory/held_results/measurement_serializer'
+require 'benchmark/memory/report/entry'
 
 module Benchmark
   module Memory
@@ -14,8 +16,8 @@ module Benchmark
         # @return [Report::Entry]
         def load(hash)
           @object = Report::Entry.new(
-            hash["item"],
-            MeasurementSerializer.load(hash["measurement"])
+            hash['item'],
+            MeasurementSerializer.load(hash['measurement'])
           )
           self
         end
@@ -25,8 +27,8 @@ module Benchmark
         # @return [Hash] The entry as a Hash.
         def to_h
           {
-            :item        => object.label,
-            :measurement => MeasurementSerializer.new(object.measurement).to_h,
+            item: object.label,
+            measurement: MeasurementSerializer.new(object.measurement).to_h
           }
         end
       end

data/lib/benchmark/memory/held_results/measurement_serializer.rb

@@ -1,6 +1,8 @@
-require "benchmark/memory/held_results/serializer"
-require "benchmark/memory/held_results/metric_serializer"
-require "benchmark/memory/measurement"
+# frozen_string_literal: true
+
+require 'benchmark/memory/held_results/serializer'
+require 'benchmark/memory/held_results/metric_serializer'
+require 'benchmark/memory/measurement'
 
 module Benchmark
   module Memory
@@ -14,9 +16,9 @@ module Benchmark
         # @return [Measurement]
         def load(hash)
           @object = Measurement.new(
-            :memory => MetricSerializer.load(hash["memory"]),
-            :objects => MetricSerializer.load(hash["objects"]),
-            :strings => MetricSerializer.load(hash["strings"])
+            memory: MetricSerializer.load(hash['memory']),
+            objects: MetricSerializer.load(hash['objects']),
+            strings: MetricSerializer.load(hash['strings'])
           )
           self
         end
@@ -26,9 +28,9 @@ module Benchmark
         # @return [Hash] The measurement as a Hash.
         def to_h
           {
-            :memory  => MetricSerializer.new(object.memory).to_h,
-            :objects => MetricSerializer.new(object.objects).to_h,
-            :strings => MetricSerializer.new(object.strings).to_h,
+            memory: MetricSerializer.new(object.memory).to_h,
+            objects: MetricSerializer.new(object.objects).to_h,
+            strings: MetricSerializer.new(object.strings).to_h
           }
         end
       end

data/lib/benchmark/memory/held_results/metric_serializer.rb

@@ -1,5 +1,7 @@
-require "benchmark/memory/held_results/serializer"
-require "benchmark/memory/measurement/metric"
+# frozen_string_literal: true
+
+require 'benchmark/memory/held_results/serializer'
+require 'benchmark/memory/measurement/metric'
 
 module Benchmark
   module Memory
@@ -14,9 +16,9 @@ module Benchmark
         #
         def load(hash)
           @object = Measurement::Metric.new(
-            hash["type"],
-            hash["allocated"],
-            hash["retained"]
+            hash['type'],
+            hash['allocated'],
+            hash['retained']
           )
           self
         end
@@ -26,9 +28,9 @@ module Benchmark
         # @return [Hash] The metric as a Hash.
         def to_h
           {
-            :allocated => object.allocated,
-            :retained  => object.retained,
-            :type      => object.type,
+            allocated: object.allocated,
+            retained: object.retained,
+            type: object.type
           }
         end
       end

data/lib/benchmark/memory/held_results/serializer.rb

@@ -1,4 +1,6 @@
-require "json"
+# frozen_string_literal: true
+
+require 'json'
 
 module Benchmark
   module Memory
@@ -33,9 +35,9 @@ module Benchmark
         # @raise [NotImplementedError]
         #   If the inheriting subclass didn't implement.
         def load(_hash)
-          fail(
+          raise(
             NotImplementedError,
-            "You must implement a concrete version in a subclass"
+            'You must implement a concrete version in a subclass'
           )
         end
 
@@ -45,19 +47,19 @@ module Benchmark
         # @raise [NotImplementedError]
         #   If the inheriting subclass didn't implement.
         def to_h
-          fail(
+          raise(
             NotImplementedError,
-            "You must implement a concrete version in a subclass"
+            'You must implement a concrete version in a subclass'
           )
         end
 
         # Convert the object to a JSON document.
         #
         # @return [String] The object as a JSON document.
-        def to_json
-          JSON.generate(to_h)
+        def to_json(*args)
+          JSON.generate(to_h, *args)
         end
-        alias_method :to_s, :to_json
+        alias to_s to_json
       end
     end
   end

data/lib/benchmark/memory/held_results.rb

@@ -1,5 +1,7 @@
-require "forwardable"
-require "benchmark/memory/held_results/entry_serializer"
+# frozen_string_literal: true
+
+require 'forwardable'
+require 'benchmark/memory/held_results/entry_serializer'
 
 module Benchmark
   module Memory
@@ -30,7 +32,7 @@ module Benchmark
       #
       # @return [void]
       def add_result(entry)
-        with_hold_file("a") do |file|
+        with_hold_file('a') do |file|
           file.write EntrySerializer.new(entry)
           file.write "\n"
         end
@@ -43,7 +45,7 @@ module Benchmark
         if @path.is_a?(String)
           File.exist?(@path)
         else
-          @path.size > 0 # rubocop:disable Style/ZeroLengthPredicate
+          @path.size.positive?
         end
       end
 
@@ -51,9 +53,7 @@ module Benchmark
       #
       # @return [void]
       def cleanup
-        if @path.is_a?(String) && File.exist?(@path)
-          File.delete(@path)
-        end
+        File.delete(@path) if @path.is_a?(String) && File.exist?(@path)
       end
 
       # Check whether to hold results.
@@ -81,9 +81,7 @@ module Benchmark
         results = with_hold_file do |file|
           file.map { |line| EntrySerializer.load(line) }
         end
-        @results = Hash[results.map do |result|
-          [result.label, result.measurement]
-        end]
+        @results = results.map { |result| [result.label, result.measurement] }.to_h
       end
 
       private
@@ -94,13 +92,11 @@ module Benchmark
       # @param _block [Proc] The block to execute on each line of the file.
       #
       # @return [void]
-      def with_hold_file(access_mode = "r", &_block)
+      def with_hold_file(access_mode = 'r', &block)
         return unless @path
 
         if @path.is_a?(String)
-          File.open(@path, access_mode) do |f|
-            yield f
-          end
+          File.open(@path, access_mode, &block)
         else
           yield @path
         end

data/lib/benchmark/memory/helpers.rb

@@ -1,3 +1,7 @@
+# frozen_string_literal: true
+
+require 'benchmark/memory/human_readable_unit'
+
 module Benchmark
   module Memory
     # Helper methods for formatting output.
@@ -23,24 +27,11 @@ module Benchmark
       #
       # @return [String] The scaled value.
       def scale(value)
-        scale = Math.log10(value)
-        scale = 0 if scale.infinite?
-        scale = (scale / 3).to_i
-        suffix =
-          case scale
-          when 1 then "k"
-          when 2 then "M"
-          when 3 then "B"
-          when 4 then "T"
-          when 5 then "Q"
-          else
-            scale = 0
-            " "
-          end
+        value = HumanReadableUnit.new(value)
 
-        format("%10.3f#{suffix}", value.to_f / (1000**scale))
+        format("%10.3f#{value.unit}", value.to_f / (1000**value.scale))
       end
-      module_function :scale
+      module_function :scale # rubocop:disable Style/AccessModifierDeclarations
     end
   end
 end

data/lib/benchmark/memory/human_readable_unit.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+require 'delegate'
+
+module Benchmark
+  module Memory
+    # Transforms raw numbers into a human-readable scale and suffix
+    class HumanReadableUnit < SimpleDelegator
+      # @return [Integer] the exponential scale of the value
+      def scale
+        scale = Math.log10(__getobj__)
+        scale = 0 if scale.infinite?
+        scale = (scale / 3).to_i
+
+        if scale <= 5
+          scale
+        else
+          0
+        end
+      end
+
+      # @return [String] the single-character unit for the value
+      def unit
+        case scale
+        when 1 then 'k'
+        when 2 then 'M'
+        when 3 then 'B'
+        when 4 then 'T'
+        when 5 then 'Q'
+        else ' '
+        end
+      end
+    end
+  end
+end

data/lib/benchmark/memory/job/io_output/comparison_formatter.rb

@@ -1,5 +1,7 @@
-require "benchmark/memory/helpers"
-require "benchmark/memory/job/io_output/metric_formatter"
+# frozen_string_literal: true
+
+require 'benchmark/memory/helpers'
+require 'benchmark/memory/job/io_output/metric_formatter'
 
 module Benchmark
   module Memory
@@ -23,7 +25,7 @@ module Benchmark
           #
           # @return [String]
           def to_s
-            return "" unless comparison.possible?
+            return '' unless comparison.possible?
 
             output = StringIO.new
             best, *rest = comparison.entries
@@ -41,27 +43,27 @@ module Benchmark
           private
 
           def add_best_summary(best, output)
-            output << summary_message("%20s: %10i allocated\n", best)
+            output << summary_message("%20s: %10i %s\n", best)
           end
 
           def add_comparison(entry, best, output)
-            output << summary_message("%20s: %10i allocated - ", entry)
+            output << summary_message('%20s: %10i %s - ', entry)
             output << comparison_between(entry, best)
             output << "\n"
           end
 
           def comparison_between(entry, best)
-            ratio = entry.allocated_memory.to_f / best.allocated_memory.to_f
+            ratio = entry.compared_metric(comparison).to_f / best.compared_metric(comparison)
 
             if ratio.abs > 1
-              format("%.2fx more", ratio)
+              format('%<ratio>.2fx more', ratio: ratio)
             else
-              "same"
+              'same'
             end
           end
 
           def summary_message(message, entry)
-            format(message, entry.label, entry.allocated_memory)
+            format(message, entry.label, entry.compared_metric(comparison), comparison.value)
           end
         end
       end

data/lib/benchmark/memory/job/io_output/entry_formatter.rb

@@ -1,6 +1,8 @@
-require "stringio"
-require "benchmark/memory/helpers"
-require "benchmark/memory/job/io_output/metric_formatter"
+# frozen_string_literal: true
+
+require 'stringio'
+require 'benchmark/memory/helpers'
+require 'benchmark/memory/job/io_output/metric_formatter'
 
 module Benchmark
   module Memory
@@ -26,11 +28,14 @@ module Benchmark
           def to_s
             output = StringIO.new
             output << rjust(entry.label)
-            entry.measurement.each_with_index.map do |metric, index|
-              output << " " * 20 unless index == 0
-              output << MetricFormatter.new(metric)
-              output << "\n"
+
+            first, *rest = *entry.measurement
+
+            output << "#{MetricFormatter.new(first)}\n"
+            rest.each do |metric|
+              output << "#{' ' * 20}#{MetricFormatter.new(metric)}\n"
             end
+
             output.string
           end
         end