From 5824981d98a61b071c216749c7dcdd40fdf61673 Mon Sep 17 00:00:00 2001 From: bronzdoc Date: Sun, 28 Jul 2019 08:06:01 -0600 Subject: [rubygems/rubygems] Remove extension 'double pipe equals' assigment since is not needed anymore https://github.com/rubygems/rubygems/commit/930de86a24 --- lib/rubygems/ext/builder.rb | 13 ------------- 1 file changed, 13 deletions(-) (limited to 'lib/rubygems/ext') diff --git a/lib/rubygems/ext/builder.rb b/lib/rubygems/ext/builder.rb index ad9fa2d506..72a2b349f0 100644 --- a/lib/rubygems/ext/builder.rb +++ b/lib/rubygems/ext/builder.rb @@ -151,19 +151,6 @@ EOF def build_extension(extension, dest_path) # :nodoc: results = [] - # FIXME: Determine if this line is necessary and, if so, why. - # Notes: - # 1. As far as I can tell, this method is only called by +build_extensions+. - # 2. The existence of this line implies +extension+ is, or previously was, - # sometimes +false+ or +nil+. - # 3. #1 and #2 combined suggests, but does not confirm, that - # +@specs.extensions+ sometimes contained +false+ or +nil+ values. - # 4. Nothing seems to explicitly handle +extension+ being empty, - # which makes me wonder both what it should do and what it does. - # - # - @duckinator - extension ||= '' # I wish I knew why this line existed - builder = builder_for(extension) extension_dir = -- cgit v1.2.3