summaryrefslogtreecommitdiffhomepage
path: root/lib/axlsx/workbook/worksheet
diff options
context:
space:
mode:
authorGeremia Taglialatela <[email protected]>2023-05-23 01:29:34 +0200
committerGeremia Taglialatela <[email protected]>2023-05-31 09:58:46 +0200
commit86026e2f106cb982f58f5a876756507d051aee4b (patch)
tree192cb431e2f5af9323d32466709e7305b48520fb /lib/axlsx/workbook/worksheet
parent3aa92cd2be4b535d1b21d2ba02f0e7660ccc6a05 (diff)
downloadcaxlsx-86026e2f106cb982f58f5a876756507d051aee4b.tar.gz
caxlsx-86026e2f106cb982f58f5a876756507d051aee4b.zip
Fix Style/SymbolProc offenses
This also provides a performance improvement ``` Comparison (array of 4 elements): map(&): 3643131.4 i/s map {}: 3488052.5 i/s - 1.04x (± 0.00) slower Comparison (array of 20 elements): map(&): 466013.9 i/s map {}: 408447.2 i/s - 1.14x (± 0.00) slower ```
Diffstat (limited to 'lib/axlsx/workbook/worksheet')
-rw-r--r--lib/axlsx/workbook/worksheet/worksheet.rb2
-rw-r--r--lib/axlsx/workbook/worksheet/worksheet_hyperlinks.rb2
2 files changed, 2 insertions, 2 deletions
diff --git a/lib/axlsx/workbook/worksheet/worksheet.rb b/lib/axlsx/workbook/worksheet/worksheet.rb
index aaeadb11..1ece3138 100644
--- a/lib/axlsx/workbook/worksheet/worksheet.rb
+++ b/lib/axlsx/workbook/worksheet/worksheet.rb
@@ -760,7 +760,7 @@ module Axlsx
raise ArgumentError, (ERR_SHEET_NAME_CHARACTER_FORBIDDEN % name) if WORKSHEET_NAME_FORBIDDEN_CHARS.any? { |char| name.include? char }
name = Axlsx::coder.encode(name)
- sheet_names = @workbook.worksheets.reject { |s| s == self }.map { |s| s.name }
+ sheet_names = @workbook.worksheets.reject { |s| s == self }.map(&:name)
raise ArgumentError, (ERR_DUPLICATE_SHEET_NAME % name) if sheet_names.include?(name)
end
diff --git a/lib/axlsx/workbook/worksheet/worksheet_hyperlinks.rb b/lib/axlsx/workbook/worksheet/worksheet_hyperlinks.rb
index f19630f8..334b59da 100644
--- a/lib/axlsx/workbook/worksheet/worksheet_hyperlinks.rb
+++ b/lib/axlsx/workbook/worksheet/worksheet_hyperlinks.rb
@@ -24,7 +24,7 @@ module Axlsx
def relationships
return [] if empty?
- map { |hyperlink| hyperlink.relationship }
+ map(&:relationship)
end
# seralize the collection of hyperlinks